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

List:       kde-commits
Subject:    kdelibs/kdecore
From:       David Faure <faure () kde ! org>
Date:       2005-03-23 20:49:14
Message-ID: 20050323204914.097493CF () office ! kde ! org
[Download RAW message or body]

CVS commit by faure: 

Testcase: "kontact --module kmail", then "kmail faure@kde.org"
 -> since the kmail options have been loaded (by the kontact process)
after a KCmdLineArgs::reset(), kontact barfs with "no such option --module".

The problem is that KCmdLineArgs::parsedArgs() reparses the real args
(argc/argv), but the whole point of calling loadAppArgs() was to ship over
the already-parsed args from the other process. This fixes it.
Approved by Waldo (and indeed when KUniqueApplication calls loadAppArgs, 
parsed is already true, so no-op there).

Bug reported by Tobias, who will close the appropriate bug report[s] :)


  M +1 -0      kcmdlineargs.cpp   1.102


--- kdelibs/kdecore/kcmdlineargs.cpp  #1.101:1.102
@@ -303,4 +303,5 @@ KCmdLineArgs::loadAppArgs( QDataStream &
      }
    }
+   parsed = true;
 }
 


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