BrushStoreWindow: fix broken check for duplicate brushes

dsizzle · dsizzle · commit 7df70c72751c · 2023-10-04T03:22:51.000-07:00
- break statement was erroneously moved during The Big Cleanup, and it
allowed duplicate brushes to be added to Brush window.
diff --git a/artpaint/windows/BrushStoreWindow.cpp b/artpaint/windows/BrushStoreWindow.cpp
@@ -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) {

Original file line number	Diff line number	Diff line change
`@@ -493,7 +493,8 @@ BrushStoreView::AddBrush(Brush* brush)`
`493`	`493`	`brush_info item = (brush_info)brush_data->ItemAt(i);`
`494`	`494`	`if (Brush::compare_brushes(item, new_brush_info) == true) {`
`495`	`495`	`index = i;`
`496`		`- } break;`
	`496`	`+ break;`
	`497`	`+ }`
`497`	`498`	`}`
`498`	`499`
`499`	`500`	`if (index < 0) {`