elementaryart team mailing list archive

[Vadim Rutkovsky <roignac@xxxxxxxxx>]

Vadim Rutkovsky has proposed merging lp:~roignac/marlin/fix-801100-double_ctrl_l into lp:marlin.

Requested reviews:
  xapantu (xapantu)
Related bugs:
  Bug #801100 in Marlin: "Crash when pressing Ctrl+L when two windows are open"
  https://bugs.launchpad.net/marlin/+bug/801100

For more details, see:
https://code.launchpad.net/~roignac/marlin/fix-801100-double_ctrl_l/+merge/66688

Fix for the related issue - widgets should not be updated if state is already set
-- 
https://code.launchpad.net/~roignac/marlin/fix-801100-double_ctrl_l/+merge/66688
Your team The elementary Project is subscribed to branch lp:marlin.

=== modified file 'libwidgets/LocationBar.vala'
--- libwidgets/LocationBar.vala	2011-06-24 23:26:10 +0000
+++ libwidgets/LocationBar.vala	2011-07-02 11:31:20 +0000
@@ -31,7 +31,12 @@
         public bool state
         {
             get { return _state; }
-            set { _state = value; update_widget(); }
+            set {
+                if (_state != value) {
+                    _state = value;
+                    update_widget();
+                }
+            }
         }
 
         public new string path{