[PATCH] KProcess port of khelpcenter

[Oswald Buddenhagen]

On Thu, Dec 20, 2007 at 04:39:51PM +0100, Ralf Habacker wrote:
> +    KProcess *proc = new KProcess();
> +	*proc << KShell::splitArgs(commonSearchProgram);
>  
tab alert! :)

> -    mSearchResult = mSearchResult.replace(QLatin1String("http://localhost/"), QLatin1String("file:/"));
> +    mSearchResult = mSearchResult.replace("http://localhost/", "file:/");
>
fwiw, the assingment is not necessary, as replace() modifies the original
string.
to be pedandic, the replacement string should be file:/// (at least some
say so ... i find it stupid myself).

>      mSearchResult = mSearchResult.mid( mSearchResult.indexOf( '<' ) );
>  
one might want to change that to remove()ing the leading chars instead.
then the assignment goes away as well. :)

rest seems fine.

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Confusion, chaos, panic - my work here is done.