that segfaulted. oops. added a workaround so setImageDither doesnt reconfig when...

diff --git a/src/Screen.cc b/src/Screen.cc
index b07f25f176ed8746df4296ade30d85ddafd810a0..e7f6f6f3f8c6f5ac0b8fd63c38e1edd5c7a194ae 100644 (file)
--- a/src/Screen.cc
+++ b/src/Screen.cc
@@ -850,13 +850,14 @@ void BScreen::setAutoRaise(bool a) {
 }
 
 
-void BScreen::setImageDither(bool d) {
+void BScreen::setImageDither(bool d, bool reconfig) {
   resource.image_dither = d;
   std::ostrstream s;
   s << "session.screen" << getScreenNumber() << ".imageDither" << ends;
   config.setValue(s.str(), resource.image_dither);
   s.rdbuf()->freeze(0);
-  openbox.reconfigure();
+  if (reconfig)
+    openbox.reconfigure();
 }
 
 
@@ -1042,7 +1043,7 @@ void BScreen::saveWorkspaceNames() {
 void BScreen::save() {
   setSloppyFocus(resource.sloppy_focus);
   setAutoRaise(resource.auto_raise);
-  setImageDither(resource.image_dither);
+  setImageDither(resource.image_dither, false);
   setOpaqueMove(resource.opaque_move);
   setFullMax(resource.full_max);
   setFocusNew(resource.focus_new);

diff --git a/src/Screen.h b/src/Screen.h
index 8acae7ee0b7d4e0947af7541da3a15e07b3e1269..99fbf40210dc0f924291d673350964affd4aa67a 100644 (file)
--- a/src/Screen.h
+++ b/src/Screen.h
@@ -219,7 +219,7 @@ public:
   void setAutoRaise(bool a);
   
   inline bool imageDither() const { return resource.image_dither; }
-  void setImageDither(bool d);
+  void setImageDither(bool d, bool reconfig = true);
   
   inline bool orderedDither() const { return resource.ordered_dither; }