do not trigger visual changes when Fill::set_* methods don't change anything
authorPaul Davis <paul@linuxaudiosystems.com>
Fri, 26 Apr 2013 18:19:16 +0000 (14:19 -0400)
committerPaul Davis <paul@linuxaudiosystems.com>
Fri, 26 Apr 2013 18:19:16 +0000 (14:19 -0400)
libs/canvas/fill.cc

index 339549f8bed936f90dba058d88f6d5072b6e83a7..9947415cf9462f433750dac108cbe9b84bb8c91e 100644 (file)
@@ -39,21 +39,21 @@ Fill::Fill (Group* parent)
 void
 Fill::set_fill_color (Color color)
 {
-       begin_visual_change ();
-       
-       _fill_color = color;
-
-       end_visual_change ();
+       if (_fill_color != color) {
+               begin_visual_change ();
+               _fill_color = color;
+               end_visual_change ();
+       }
 }
 
 void
 Fill::set_fill (bool fill)
 {
-       begin_visual_change ();
-       
-       _fill = fill;
-
-       end_visual_change ();
+       if (_fill != fill) {
+               begin_visual_change ();
+               _fill = fill;
+               end_visual_change ();
+       }
 }
 
 void