[Geany-devel] Small additions to stash

Dimitar Zhekov dimitar.zhekov at xxxxx
Thu Apr 5 18:02:27 UTC 2012


On Thu, 05 Apr 2012 17:20:44 +0100
Nick Treleaven <nick.treleaven at btinternet.com> wrote:

> On 04/04/2012 19:58, Dimitar Zhekov wrote:
> > Hi,
> >
> > How about some $subject? Namely:
> >
> > Include stash_group_set_use_defaults() in plugin StashFuncs. I'm not
> > using it any more, but it's a nice thing to have.
> 
> I'm not convinced we should support that publicly. It puts a burden on 
> the implementation for an unusual use case.

ACK.

> > 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.

-- 
E-gards: Jimmy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-A-stash_group_free_settings-function.patch
Type: text/x-diff
Size: 3038 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20120405/de7dcb3c/attachment.patch>


More information about the Devel mailing list