GNOME Bugzilla – Bug 611707
Move documentation from templates to inline comments (GtkPlug, GtkSocket)
Last modified: 2010-04-18 00:50:30 UTC
As specified at http://live.gnome.org/GTK%2B/TaskAPIDocMigration
Created attachment 155139 [details] [review] Move GtkPlug documentation to inline comments
Created attachment 155140 [details] [review] Move GtkSocket documentation to inline comments
Review of attachment 155140 [details] [review]: Thank you for your patch. Could you fix the issues commented below? Also, I'd suggest you to use the git format to send the patches: take a look here for a little guide: http://live.gnome.org/GnomeLove/SubmittingPatches ::: gtk/gtksocket.c @@ +55,3 @@ + * @Short_description: Container for widgets from other processes + * SECTION:gtksocket +/** Don't use a <varlistentry> in @See_also:, replace with a simple list (see gtk/gtkfilechooser.c for example) @@ +96,3 @@ + * has been created by examining the + * <structfield>plug_window</structfield> field of the + * #GtkSocket structure. If this field is non-%NULL, You should rephrase this to use gtk_socket_get_plug_window() instead access the structure field directly. Something like "You can check if the plug has been created by using gtk_socket_get_plug_window()" ::: gtk/gtksocket.h @@ +52,3 @@ + * + * GtkSocket: +/** Now in GTK+ all object structures are opaque, so there is no need to documenting them
Comment on attachment 155139 [details] [review] Move GtkPlug documentation to inline comments commit 7ab2155aef8e2069a9e5680ca2cd68f70eec43fd
Created attachment 155368 [details] [review] Move GtkSocket documentation to inline comments
Ping ?
Sorry for the delay Patrick, the patch looks good, but the 2.24 release is inminent so no big changes in the code are allowed. I'll try to apply your patch as soon as 2.20 is released. Thank you!
Comment on attachment 155368 [details] [review] Move GtkSocket documentation to inline comments commit 6a9b72525ac5324897693d564dd283e50b58c222
This problem has been fixed in the development version. The fix will be available in the next major software release. Thanks Patrick!