[Github-comments] [geany/geany] api: correctly document ownership transfer (#1125)
Thomas Martitz
notifications at xxxxx
Thu Jul 7 14:51:21 UTC 2016
plugin_set_key_group() and _full() effectively take the ownership,
because the section_name is stored inside Geany.
Plugins usually pass a static string so it's not a problem. But GI-based
plugins might pass dynamic strings, so this behavior needs to be documented.
Fixes keybindings.conf being written with garbage (for a GI-based plugin).
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1125
-- Commit Summary --
* api: correctly document ownership transfer
-- File Changes --
M src/pluginutils.c (4)
-- Patch Links --
https://github.com/geany/geany/pull/1125.patch
https://github.com/geany/geany/pull/1125.diff
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1125
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160707/dfa2cf1a/attachment.html>
More information about the Github-comments
mailing list