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

List:       kde-commits
Subject:    KDE/kdelibs/kdecore/jobs
From:       Raphael Kubo da Costa <kubito () gmail ! com>
Date:       2009-04-07 21:47:15
Message-ID: 1239140835.926304.13135.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 950846 by rkcosta:

Update apidox related to kill, result and finished as it wasn't clear client code \
shouldn't connect to finished and result isn't always emitted.


 M  +12 -3     kjob.h  


--- trunk/KDE/kdelibs/kdecore/jobs/kjob.h #950845:950846
@@ -166,7 +166,8 @@
      * and ask uiserver to close the progress window.
      * @p verbosity is set to EmitResult for subjobs. Whether applications
      * should call with Quietly or EmitResult depends on whether they rely
-     * on result being emitted or not.
+     * on result being emitted or not. Please notice that if @p verbosity is
+     * set to Quietly, signal result will NOT be emitted.
      * @return true if the operation is supported and succeeded, false otherwise
      */
     bool kill( KillVerbosity verbosity = Quietly );
@@ -326,6 +327,9 @@
      * This is a private signal, it can't be emitted directly by subclasses of
      * KJob, use emitResult() instead.
      *
+     * Client code is not supposed to connect to this signal, signal result should
+     * be used instead.
+     *
      * @param job the job that emitted this signal
      * @internal
      */
@@ -352,13 +356,18 @@
     void resumed(KJob *job);
 
     /**
-     * Emitted when the job is finished, in any case (completed, canceled,
-     * failed...). Use error to know the result.
+     * Emitted when the job is finished (except when killed with KJob::Quietly).
      *
+     * Use error to know if the job was finished with error.
+     *
      * This is a private signal, it can't be emitted directly by subclasses of
      * KJob, use emitResult() instead.
      *
+     * Please connect to this signal instead of finished.
+     *
      * @param job the job that emitted this signal
+     *
+     * @see kill
      */
     void result(KJob *job);
 


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

Configure | About | News | Add a list | Sponsored by KoreLogic