Rewrite conditional, remove always-true clause

Parameter is unsigned, no need to check that it's >= 0. Closes #132
mypaint · Dec 27, 2019 · c2e720c · c2e720c
1 parent 0a26ee1
commit c2e720c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/mypaint-brush.c b/mypaint-brush.c
@@ -1437,7 +1437,7 @@ update_brush_setting_from_json_object(MyPaintBrush *self,
 {
     MyPaintBrushSetting setting_id = mypaint_brush_setting_from_cname(setting_name);
 
-    if (!(setting_id >= 0 && setting_id < MYPAINT_BRUSH_SETTINGS_COUNT)) {
+    if (setting_id >= MYPAINT_BRUSH_SETTINGS_COUNT) {
         fprintf(stderr, "Warning: Unknown setting_id: %d for setting: %s\n",
                 setting_id, setting_name);
         return FALSE;