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 673229 - glib: Use Returns:, not @returns
glib: Use Returns:, not @returns
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)
: 672783 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-03-31 09:58 UTC by Robert Ancell
Modified: 2015-02-07 16:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix return annotation (52.68 KB, patch)
2012-03-31 10:00 UTC, Robert Ancell
none Details | Review
Use "Returns:" instead of the invalid "@returns" for annotating return values (49.46 KB, patch)
2012-10-31 02:00 UTC, Robert Ancell
committed Details | Review

Description Robert Ancell 2012-03-31 09:58:10 UTC
There are a number of return values specified in the parameter format, not the correct "Returns:" or "Return value:" as specified here:
https://live.gnome.org/GObjectIntrospection/Annotations
Comment 1 Robert Ancell 2012-03-31 10:00:07 UTC
Created attachment 211026 [details] [review]
Fix return annotation
Comment 2 Johan (not receiving bugmail) Dahlin 2012-03-31 17:39:00 UTC
gobject-introspection probably needs to understand the @return format it as gtk-doc does.
Comment 3 Dieter Verfaillie 2012-04-06 13:02:39 UTC
The annotationparser now understands @returns parameter, Returns:
and Return value: tags and internally standardises on the Returns:
tag. Fixed in bug #672254.
Comment 4 Robert Ancell 2012-04-07 02:52:46 UTC
Dieter, is the recommended default still 'Returns:'?  The purpose of this patch was mostly to make the use in glib consistent and match the documentation so new annotations are done in a consistent format.
Comment 5 Emmanuele Bassi (:ebassi) 2012-04-07 22:06:58 UTC
"Returns" and "Return value" are equivalent; it's @returns that is something not really valid, even though it's accepted by gtk-doc.
Comment 6 Dieter Verfaillie 2012-04-08 09:49:58 UTC
(In reply to comment #4)
> Dieter, is the recommended default still 'Returns:'?  The purpose of this patch
> was mostly to make the use in glib consistent and match the documentation so
> new annotations are done in a consistent format.

Yeah, I misunderstood the intent of your patch and marked this bug as
resolved in reaction to comment #2...

(In reply to comment #5)
> "Returns" and "Return value" are equivalent; it's @returns that is something
> not really valid, even though it's accepted by gtk-doc.

Yes, and the patch attached below addresses exactly that concern.
Comment 7 Colin Walters 2012-10-27 17:00:36 UTC
Comment on attachment 211026 [details] [review]
Fix return annotation

Unmarking obsolete based on comments.  Can you rebase this against master?

Also the commit message could be more descriptive, like "Use Returns:, not @returns parameter"
Comment 8 Colin Walters 2012-10-30 13:30:13 UTC
*** Bug 672783 has been marked as a duplicate of this bug. ***
Comment 9 Robert Ancell 2012-10-31 02:00:21 UTC
Created attachment 227699 [details] [review]
Use "Returns:" instead of the invalid "@returns" for  annotating return values

Updated to master.
Comment 10 Colin Walters 2012-10-31 12:35:57 UTC
Review of attachment 227699 [details] [review]:

Looks good.  

Remember to add a reference to this bug in the commit message per the style guide from https://live.gnome.org/GnomeLove/SubmittingPatches
Comment 11 André Klapper 2015-02-07 16:49:17 UTC
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]