[Geany-devel] Small additions to stash

Dimitar Zhekov dimitar.zhekov at xxxxx
Fri Apr 6 20:04:45 UTC 2012


On Fri, 06 Apr 2012 14:06:48 +0100
Nick Treleaven <nick.treleaven at btinternet.com> wrote:

> On 05/04/2012 19:02, Dimitar Zhekov wrote:
> >>> New stash_group_free_strings(), which frees any strings and string
> >>> >  >  arrays in a group. Much easier than to track them individually, as
> >>> >  >  in Geany (though if someone is willing to track the keyfile_groups
> >>> >  >  prefs free-s and remove them, a call may be included in
> >>> >  >  configuration_finalize).
> >> >
> >> >  Probably it should be named more generally, in case we support e.g.
> >> >  null-terminated integer lists. Maybe stash_group_free_settings().
> > Renamed and changed the description. Updated patch attached.
> 
> Thanks, applied. I added a note to the docs that it's not called by 
> stash_group_free().
> 
> Ideally we would be using the function in Geany somewhere as an example.

Perhaps on the plugins group, since it's short and contains both string
(freed in main?..) and string vector. Would require a static variable.

-- 
E-gards: Jimmy



More information about the Devel mailing list