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

[Ralf Habacker]

Aaron J. Seigo schrieb:
> On Friday 28 March 2008, Jarosław Staniek wrote:
>   
>> 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
>>     
>
> +1
>
>   
>> 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().
>>     
>
> will leave for win32 devs to comment on =)
>   
I suggest to allow kde internal (konqueror) and external browser 
switchable by a configuration switch. Would be kdeglobals a good place 
for such a switch ?

Ralf