[Geany-devel] small svn Goto Line dialog fix

Enrico Tröger enrico.troeger at xxxxx
Mon May 10 17:36:57 UTC 2010


On Mon, 10 May 2010 13:23:28 +0300, Dimitar wrote:

> Hi.
> 
> In geany-svn, in the Goto Line dialog, Cancel works the same way as
> OK. That's because on_go_to_line_activate() checks the result of
> dialogs_show_input_goto_line() for NULL, but the result is always a
> copy of either the new or the old/default text. Altering
> dialogs_show_input_goto_line() not to use NVL() seems to work.

Oops, copy&paste mistake. Still, I'm wondering a bit why I couldn't
reproduce it on my system though by reading the code it must had
happened as you described. Weird, maybe it's because of my compile
flags.


> While on that, why not make dialog_input static? It should never be
> used outside dialogs.

Thanks, both changes committed.

Regards,
Enrico

-- 
Get my GPG key from http://www.uvena.de/pub.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20100510/659a4ab9/attachment.pgp>


More information about the Devel mailing list