Skip to content

Commit

Permalink
BrushStoreWindow: fix broken check for duplicate brushes
Browse files Browse the repository at this point in the history
- break statement was erroneously moved during The Big Cleanup, and it
allowed duplicate brushes to be added to Brush window.
  • Loading branch information
dsizzle committed Oct 4, 2023
1 parent 31949a9 commit 4155988
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions artpaint/windows/BrushStoreWindow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
#include <ScrollBar.h>
#include <ScrollView.h>


s
#undef B_TRANSLATION_CONTEXT
#define B_TRANSLATION_CONTEXT "Windows"

Expand Down Expand Up @@ -493,7 +493,8 @@ BrushStoreView::AddBrush(Brush* brush)
brush_info item = *(brush_info*)brush_data->ItemAt(i);
if (Brush::compare_brushes(item, new_brush_info) == true) {
index = i;
} break;
break;
}
}

if (index < 0) {
Expand Down

0 comments on commit 4155988

Please sign in to comment.