[Github-comments] [geany/geany] Use a format string instead of repeating almost the same string 9 times (#1058)

Colomban Wendling notifications at xxxxx
Wed Jun 8 22:41:53 UTC 2016


This simplifies translations by being less redundant.

Note: label variable is not freed because `keybindings_set_item()` currently assumes static strings but for plugins, so we need not free the label right away.  It's not a real problem though, as this string
is required during the whole process' lifetime, so it'd use just as much memory anyway.

Closes #953.

---
I'm not certain this is a great way to fix, but it works.  Opinions welcome.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/1058

-- Commit Summary --

  * Use a format string instead of repeating almost the same string 9 times

-- File Changes --

    M src/keybindings.c (39)

-- Patch Links --

https://github.com/geany/geany/pull/1058.patch
https://github.com/geany/geany/pull/1058.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/1058
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160608/55cc0547/attachment.html>


More information about the Github-comments mailing list