Am Di Dezember 27 2005 12:17 schrieb David Faure: > The idea is fine; the patch duplicates the adding of the url, why not > do this instead? > > - cmd += " " + KProcess::quote(url.url()) + " " + > KProcess::quote(suggestedFilename); + cmd += " " + > KProcess::quote(url.url()); > + if ( !suggestedFilename.isEmpty() ) > + cmd += " " + KProcess::quote(suggestedFilename); Yes, this is better, I'll commit tomorrow (gotta go) Rainer >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<