note that this patch does not change GI annotations so it can be applied safely
LGTM with questions https://codereview.appspot.com/5645095/diff/2001/src/ibusconfig.h File src/ibusconfig.h (right): https://codereview.appspot.com/5645095/diff/2001/src/ibusconfig.h#newcode138 src/ibusconfig.h:138: * See also: ibus_config_set_value(). @see_also is a keyword for gtk-doc http://developer.gnome.org/gtk-doc-manual/unstable/documenting_sections.html.en You changed it to See also, does it work?
https://codereview.appspot.com/5645095/diff/2001/src/ibusconfig.h File src/ibusconfig.h (right): https://codereview.appspot.com/5645095/diff/2001/src/ibusconfig.h#newcode138 src/ibusconfig.h:138: * See also: ibus_config_set_value(). On 2012/02/13 17:15:37, Peng wrote: > @see_also is a keyword for gtk-doc > http://developer.gnome.org/gtk-doc-manual/unstable/documenting_sections.html.en > > You changed it to See also, does it work? @see_also is only allowed in section comment block not in function comment block: http://developer.gnome.org/gtk-doc-manual/unstable/documenting_symbols.html.en So, I get "see_also: " in the output HTML if @see_also is used. @returns, which is also not supported in function comment block, still seems to work though.
On 2012/02/14 01:21:13, Daiki Ueno wrote: > https://codereview.appspot.com/5645095/diff/2001/src/ibusconfig.h > File src/ibusconfig.h (right): > > https://codereview.appspot.com/5645095/diff/2001/src/ibusconfig.h#newcode138 > src/ibusconfig.h:138: * See also: ibus_config_set_value(). > On 2012/02/13 17:15:37, Peng wrote: > > @see_also is a keyword for gtk-doc > > > http://developer.gnome.org/gtk-doc-manual/unstable/documenting_sections.html.en > > > > You changed it to See also, does it work? > > @see_also is only allowed in section comment block not in function comment > block: > http://developer.gnome.org/gtk-doc-manual/unstable/documenting_symbols.html.en > So, I get "see_also: " in the output HTML if @see_also is used. @returns, which > is also not supported in function comment block, still seems to work though. I see. thanks for explanation. lgtm