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 660352 - g-ir-scanner warns about parameter even though (skip)
g-ir-scanner warns about parameter even though (skip)
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)
Depends on:
Blocks:
 
 
Reported: 2011-09-28 11:39 UTC by Stef Walter
Modified: 2015-02-07 17:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Skip analysis of params that have been (skip)'d (907 bytes, patch)
2011-09-28 12:02 UTC, Stef Walter
reviewed Details | Review

Description Stef Walter 2011-09-28 11:39:23 UTC
When building the gnome-keyring gir for the Gck library, we get this warning:

gck-object.c:955: Warning: Gck: gck_object_get_data_async: argument allocator: Missing (scope) annotation for callback without GDestroyNotify (valid: call, async)

Even though the parameter in question has:

/**
 * gck_object_get_data_async:
 * @self: The object to get attribute data from.
 * @attr_type: The attribute to get data for.
 * @allocator: (skip): An allocator with which to allocate memory for the data, or NULL for default.
 * @cancellable: Optional cancellation object, or NULL.
 * @callback: Called when the operation completes.
 * @user_data: Data to be passed to the callback.
...
Comment 1 Stef Walter 2011-09-28 12:02:27 UTC
Created attachment 197652 [details] [review]
Skip analysis of params that have been (skip)'d

Otherwise we get warnings for params that have been skipped
Comment 2 Johan (not receiving bugmail) Dahlin 2011-09-28 14:15:59 UTC
Review of attachment 197652 [details] [review]:

This looks good, but be sure that make check passes before this can be submitted.
Comment 3 Stef Walter 2011-09-29 08:42:26 UTC
All checks passed. Thanks. Merged into master.
Comment 4 André Klapper 2015-02-07 17:00:50 UTC
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]