[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