After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 743879 - special case G_DECLARE_*_TYPE
special case G_DECLARE_*_TYPE
Status: RESOLVED FIXED
Product: gtk-doc
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: 1.22
Assigned To: gtk-doc maintainers
gtk-doc maintainers
Depends on:
Blocks:
 
 
Reported: 2015-02-02 17:56 UTC by Xavier Claessens
Modified: 2015-03-13 09:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
scanner: special case G_DECLARE_*_TYPE (1.39 KB, patch)
2015-02-02 18:52 UTC, Xavier Claessens
none Details | Review
scanner: special case G_DECLARE_*_TYPE (1.48 KB, patch)
2015-02-02 22:00 UTC, Xavier Claessens
none Details | Review
scanner: special case G_DECLARE_* (2.08 KB, patch)
2015-02-03 15:37 UTC, Xavier Claessens
none Details | Review
scanner: special case G_DECLARE_* (2.08 KB, patch)
2015-02-03 16:38 UTC, Xavier Claessens
none Details | Review

Description Xavier Claessens 2015-02-02 17:56:46 UTC
GLib master has new macro G_DECLARE_FINAL_TYPE and G_DECLARE_DERIVABLE_TYPE. I would imaging that G_DECLARE_INTERFACE_TYPE could be added later. They break gtkdoc-scan parsing.

For example gliststore.h contains this:

GLIB_AVAILABLE_IN_2_44
G_DECLARE_FINAL_TYPE(GListStore, g_list_store, G, LIST_STORE, GObject)

gtkdoc-parse think that's the declaration of a function named "G_DECLARE_FINAL_TYPE" with return type "GLIB_AVAILABLE_IN_2_44".

When ran with --rebuild-types it also won't see the _get_type() function.

I think that should be special cased.
Comment 1 Xavier Claessens 2015-02-02 18:52:50 UTC
Created attachment 295968 [details] [review]
scanner: special case G_DECLARE_*_TYPE
Comment 2 Xavier Claessens 2015-02-02 22:00:19 UTC
Created attachment 295982 [details] [review]
scanner: special case G_DECLARE_*_TYPE
Comment 3 Xavier Claessens 2015-02-03 15:37:58 UTC
Created attachment 296034 [details] [review]
scanner: special case G_DECLARE_*
Comment 4 Xavier Claessens 2015-02-03 15:42:14 UTC
Note that it already support G_DECLARE_INTERFACE as proposed on bug #743939
Comment 5 Xavier Claessens 2015-02-03 16:38:57 UTC
Created attachment 296042 [details] [review]
scanner: special case G_DECLARE_*
Comment 6 Xavier Claessens 2015-02-05 19:49:39 UTC
Code pushed as wip/xclaesse/fixes-for-glib
Comment 7 Stefan Sauer (gstreamer, gtkdoc dev) 2015-03-13 09:45:05 UTC
The following fix has been pushed:
25d07c2 scanner: special case G_DECLARE_*