On Fri, 8 Oct 2010 05:41:00 +0400
Eugene Arshinov <[email protected]> wrote:

> > The change was mainly to make editor_snippets_init() shorter for
> > easier understanding. Now system keyfile snippet keybindings are
> > supported too.
> > 
> 
> Thanks for committing and improving the patch.
> 
> These is a little flaw in current editor.c:load_kb() function.  It will
> probably segfault if snippets.conf does not contain Keybindings
> section.  You should check if (keys != NULL) before doing
> foreach_strv(ptr, keys).

foreach_strv is null safe since that commit ;-)

Regards,
Nick
_______________________________________________
Geany-devel mailing list
[email protected]
http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel

Reply via email to