Skip to content

gtk.cfg: Add G_DEPRECATED_FOR and G_ENCODE_VERSION#8708

Open
correctmost wants to merge 1 commit into
cppcheck-opensource:mainfrom
correctmost:add-glib-macros
Open

gtk.cfg: Add G_DEPRECATED_FOR and G_ENCODE_VERSION#8708
correctmost wants to merge 1 commit into
cppcheck-opensource:mainfrom
correctmost:add-glib-macros

Conversation

@correctmost

Copy link
Copy Markdown
Contributor

G_DEPRECATED_FOR is defined in glib/gmacros.h.
G_ENCODE_VERSION is defined in glib/gversionmacros.h.

G_DEPRECATED_FOR is defined in glib/gmacros.h.
G_ENCODE_VERSION is defined in glib/gversionmacros.h.
@danmar

danmar commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

I accept this change, there is no test added and therefore I assume you will make sure it will work properly.

have you been added in the AUTHORS file?

@correctmost

Copy link
Copy Markdown
Contributor Author

I accept this change, there is no test added and therefore I assume you will make sure it will work properly.

I should've mentioned that I tested the changes on https://github.com/GNOME/pango, which uses both macros.

Before this change, I would get quick failures:

pango/pango-version-macros.h:385:2 failed to evaluate #elif condition, undefined function-like macro invocation: G_ENCODE_VERSION( ... )
pango/pango-script.h:309:1 There is an unknown macro here somewhere. Configuration is required. If G_DEPRECATED_FOR is a macro then please configure it.

have you been added in the AUTHORS file?

No credit is really needed :). I'm just trying to reduce some false positives I see when checking Xfce and GNOME code.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants