[patch] KToolInvocation: re-enable Windows code, share with UNIX, add invokeTerminal()

Jarosław Staniek js at iidea.pl
Fri Mar 28 11:09:17 GMT 2008


For review.

1. The following methods are identical on all platforms and thus would be good 
to  move them to the common ktoolinvocation.cpp file.
- KToolInvocation::invokeHelp()
    identical (except and ifdef for problem with startServiceByDesktopName())
- KToolInvocation::invokeMailer() - identical
- KToolInvocation::startKdeinit() - identical except additional --suicide arg 
for UNIX

2. Reneable code that was used in KDE3/win32 (and now runs fine too)
- void KToolInvocation::invokeMailer(const QString &_to, const QString &_cc,
				  const QString &_bcc,
                                  const QString &subject, const QString &body,
                                  const QString & /*messageFile 
TODO*/, const 					 QStringList &attachURLs,
                                  const QByteArray& startup_id )
- void KToolInvocation::invokeBrowser( const QString &url, const QByteArray& 
startup_id )

Both methods use ShellExecute().

3. BTW, add Aaron's most recent KToolInvocation::invokeTerminal() for invoking 
the standard terminal application.
(TODO on Windows, will probably run cmd.exe by default)

-- 
regards / pozdrawiam, Jaroslaw Staniek
  Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
  Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/kexi)
  KDE Libraries for MS Windows (http://windows.kde.org)
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ktoolinvocation.patch
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080328/e695e733/attachment.ksh>


More information about the kde-core-devel mailing list