[Github-comments] [geany-plugins] Define G_LOG_DOMAIN for all plugins where it is missing (#373)

Enrico Tröger notifications at xxxxx
Sat Feb 27 11:04:13 UTC 2016


This helps to get proper log messages prefixed with the plugin name
in Geany's Help->Debug Messages view as well as on console
logging.

For some plugins this might be unnecessary if they don't use any `g_message()`, `g_warning()` or similar functions but `G_LOG_DOMAIN` is also used by `g_return_if_fail()` and friends.
Defining it for all plugins at least reduces the chance to miss some plugin, also for the future.

Omitted the plugins GeanyPy and GeanyPG where seperate PRs exist (#371 and #370).
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany-plugins/pull/373

-- Commit Summary --

  * Define G_LOG_DOMAIN for all plugins where it is missing

-- File Changes --

    M autoclose/src/Makefile.am (1)
    M automark/src/Makefile.am (1)
    M codenav/src/Makefile.am (1)
    M debugger/src/Makefile.am (1)
    M defineformat/src/Makefile.am (1)
    M devhelp/src/Makefile.am (1)
    M geanydoc/src/Makefile.am (2)
    M geanyextrasel/src/Makefile.am (1)
    M geanyinsertnum/src/Makefile.am (1)
    M geanylatex/src/Makefile.am (1)
    M geanylipsum/src/Makefile.am (1)
    M geanylua/Makefile.am (2)
    M geanymacro/src/Makefile.am (1)
    M geanyminiscript/src/Makefile.am (1)
    M geanynumberedbookmarks/src/Makefile.am (1)
    M geanyprj/src/Makefile.am (1)
    M geanysendmail/src/Makefile.am (1)
    M geanyvc/src/Makefile.am (1)
    M lineoperations/src/Makefile.am (1)
    M markdown/src/Makefile.am (1)
    M multiterm/src/Makefile.am (1)
    M overview/overview/Makefile.am (1)
    M pairtaghighlighter/src/Makefile.am (1)
    M pretty-printer/src/Makefile.am (1)
    M scope/src/Makefile.am (1)
    M shiftcolumn/src/Makefile.am (1)
    M tableconvert/src/Makefile.am (1)
    M treebrowser/src/Makefile.am (1)
    M xmlsnippets/src/Makefile.am (1)

-- Patch Links --

https://github.com/geany/geany-plugins/pull/373.patch
https://github.com/geany/geany-plugins/pull/373.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/373
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160227/27ac849a/attachment.html>


More information about the Github-comments mailing list