KDevelop custom tools bugfix and extension

[Harald Fernengel]

Hi,

thank you very much for your patches, I have applyed them and added your name 
to the AUTHORS file. Feel free to send in more ;) If you like we can also 
talk about CVS write access.

I escaped %W and %T using KRun::shellEscape, this works pretty well.

Harry

> Hi
>
> Bug: Tools dialog does not save arguments string.
>
> Reason: a QStrList containing empty strings is written (into config) as an
> empty string.
>
> Solution: either change QStrList (dangerous) or check for empty argument
> strings and save a dummy space character instead (patch attached). The
> dummy space character will be ignored by the shell.
>
> Extension: I'd like to integrate a custom tool that takes the currently
> selected word (in the editor) as an argument.
> I've seen there is (undocumented) %H, %S and %D macro substitution in the
> tools arguments already. I have exteded these to include the currently
> selected word (%W) and text (%T).
>
> The latter should probably be shell-escaped and put in quotes. I'm sure
> there is a lib call for exactly that (but I haven't got a clue).
>
> I've still got a wish: Add custom tools (or at least the first 5 or so) to
> the keyboard accelerator config ;-)
>
> -- Mark


-
to unsubscribe from this list send an email to kdevelop-devel-request at kdevelop.org with the following body:
unsubscribe ťyour-email-addressŤ