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 109688 - Gtk::Window::set_default() docu bug
Gtk::Window::set_default() docu bug
Status: RESOLVED FIXED
Product: gtkmm
Classification: Bindings
Component: reference documentation
2.2
Other Linux
: Normal normal
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks:
 
 
Reported: 2003-04-01 13:53 UTC by Oli Kessler
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Documentation patch (1.81 KB, patch)
2003-04-01 13:53 UTC, Oli Kessler 		none Details | Review
Documentation patch with C++ names (1.81 KB, patch)
2003-04-01 21:12 UTC, Oli Kessler 		none Details | Review

Description Oli Kessler 2003-04-01 13:53:12 UTC 
The documenation states that you use the macro GTK_WIDGET_SET_FLAGS() to
set the flag Gtk::CAN_DEFAULT for a widget you wish to use as default. 

This is wrong and will not work (it's a C macro). Instead you use the method
Gtk::Widget::set_flags() to change widget flags.

The patch changes the ChangeLog and the documentation in gtk_docs_override.xml
Comment 1 Oli Kessler 2003-04-01 13:53:55 UTC 
Created attachment 15355 [details] [review]
Documentation patch
Comment 2 Murray Cumming 2003-04-01 17:49:48 UTC 
Great. You've got the idea.

If you are overriding the text anyway, it's best to use C++ names
rather than relying on gtkmmproc to morph the C function names to C++
equivalents. So, I have to ask you whether you tested this by building
the html reference docs with it, and whether that then shows e.g.
"Gtk::Widget::set_flags()" instead of "gtk_widget_set_flags".
Comment 3 Oli Kessler 2003-04-01 21:11:38 UTC 
Right - gtkmmproc does not do the morphing (as I assumed). My fault.

Have to use the proper C++ names.
Comment 4 Oli Kessler 2003-04-01 21:12:55 UTC 
Created attachment 15365 [details] [review]
Documentation patch with C++ names
Comment 5 Murray Cumming 2003-04-02 15:34:08 UTC 
Thanks. Well done. This should be applied.
Comment 6 Murray Cumming 2003-04-10 18:13:41 UTC 
Applied. Thanks. Sorry for the delay.