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 627152 - g-ir-scanner should support .cpp files as source for introspection annotation
g-ir-scanner should support .cpp files as source for introspection annotation
Status: RESOLVED FIXED
Product: gobject-introspection
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gobject-introspection Maintainer(s)
gobject-introspection Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2010-08-17 13:33 UTC by Gustavo Noronha (kov)
Modified: 2015-02-07 16:51 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed patch (1.75 KB, patch)
2010-08-17 14:32 UTC, Gustavo Noronha (kov) 		committed Details | Review

Description Gustavo Noronha (kov) 2010-08-17 13:33:56 UTC 
In WebKitGTK+ we have our C API implemented in C++ files (.cpp). To add GObject Introspection annotations as it is now we would have to move our doc strings to the header files, which are parsed by g-ir-scanner today. That is a solution we dislike, because we would be diverging from what is usually done, and it could become quite error prone.

See https://bugs.webkit.org/show_bug.cgi?id=43054 for more information.
Comment 1 Gustavo Noronha (kov) 2010-08-17 14:32:33 UTC 
Created attachment 168086 [details] [review]
proposed patch
Comment 2 Johan (not receiving bugmail) Dahlin 2010-08-17 14:37:23 UTC 
Review of attachment 168086 [details] [review]:

Looks great thanks. Perhaps it should be explicitly mentioned that the scanner still doesn't support C++ which isn't also valid C syntax.
Comment 3 Gustavo Noronha (kov) 2010-08-17 14:44:42 UTC 
Comment on attachment 168086 [details] [review]
proposed patch

Pushed as 5e45b27c31df528672134961e4655d5fca08cbc7 with a comment, and additional explanation in the commit message about this not being real C++ parsing.
Comment 4 André Klapper 2015-02-07 16:51:39 UTC 
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]