[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE_3_1_BRANCH: kdebase/kcontrol/screensaver
From:       George Staikos <staikos () kde ! org>
Date:       2003-04-30 21:07:23
[Download RAW message or body]

CVS commit by staikos: 

backport:
Don't allow phantom screensaver processes to live on when the preview/test has finished.

Lesson of the day: don't expect KProcess::kill() to work if you use KProcess::start(KProcess::DontCare), \
instead KProcess::NotifyOnExit must be used.

Huge thanks to ossi for helping me work out what was wrong with KProcess :)


  M +1 -1      scrnsave.cpp   1.56.2.3


--- kdebase/kcontrol/screensaver/scrnsave.cpp  #1.56.2.2:1.56.2.3
@@ -769,5 +769,5 @@ void KScreenSaver::slotTest()
 
         mTesting = true;
-        mTestProc->start(KProcess::DontCare);
+        mTestProc->start(KProcess::NotifyOnExit);
     }
 }


[prev in list] [next in list] [prev in thread] [next in thread]