that segfaulted. oops. added a workaround so setImageDither doesnt reconfig when saving off all the rc values

2 files changed, 5 insertions, 4 deletions

diff --git a/src/Screen.cc b/src/Screen.cc
index b07f25f1..e7f6f6f3 100644
--- 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 8acae7ee..99fbf402 100644
--- 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; }