Fix for Bug #39538

[W. Tasin]

Hi Harry,

sorry for the late response.
Unfortunately your patch doesn't work for KDE 2, because 
QStringList::remove() is a void function.
Even multiple occurrences won't be handled. (e.g. switching on math and 
giving -lm in the text-edit field)


And another thing I would like to see: Please don't use global 
functions, use private methods instead.

I committed a patch which handles these problems.

Ciao

Walter

Harald Fernengel wrote:

>Hi,
>
>thanks John and Walter for pointing out the error in the patch, I've rewritten 
>it using a QStringList which should be fool-proof now. It uses 
>QStringList::split( " ", ldadd ) so each lib should be accessible without 
>QRegExp and Whitespace magic. After joining the StringList we shouldn't have 
>problems with multiple Whitespaces.
>
>Please review and tell me whether I can apply it...
>
>Best regards,
>Harry
>

-- 
The KDevelop project: tasin at kdevelop.de [www.kdevelop.org]
--
oohhh sveglia.... il mondo e' ammalato, ma x colpa di chi.........
(Zucchero)
:-------W. Tasin, FB 04,FHM-------------------PGP-KeyID:0x7961A645----------:
<Key-Fingerprint: 1610 835F 0080 32F4 6140  6CF7 A7D0 44CD 7961A645>