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 482158 - generate_extra_defs handels interfaces and enums wrong
generate_extra_defs handels interfaces and enums wrong
Status: RESOLVED FIXED
Product: glibmm
Classification: Bindings
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks:
 
 
Reported: 2007-10-01 13:23 UTC by 28872d13
Modified: 2007-10-05 09:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for generate_extra_defs to handle interfaces and enums correctly (1.55 KB, patch)
2007-10-01 13:24 UTC, 28872d13
none Details | Review

Description 28872d13 2007-10-01 13:23:13 UTC
If you give an interface type to get_defs() in generate_extra_defs.cc, it won´t generate the property and signal definitions, because it only parses classes. If you give it an enum type, it tries to get the signals of it and generates error messages.
Comment 1 28872d13 2007-10-01 13:24:15 UTC
Created attachment 96458 [details] [review]
Patch for generate_extra_defs to handle interfaces and enums correctly
Comment 2 Murray Cumming 2007-10-05 08:02:43 UTC
Well done. Committed to svn trunk. Sorry for not noticing this patch before now.

Please remember to patch the ChangeLog in future.
Comment 3 Sven Herzberg 2007-10-05 09:20:08 UTC
Is it possible to backport this bug fix to the stable branch?
Comment 4 Sven Herzberg 2007-10-05 09:27:49 UTC
2007-10-05  Philipp Kerling <philipp@gmail.com>

* tools/extra_defs_gen/generate_extra_defs.cc: 
Also create defs for interfaces, by refing and unrefing them 
temporarily.

Thank you.
Comment 5 Murray Cumming 2007-10-05 09:32:14 UTC
Sven, yes, go ahead if it's useful to you.