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 756251 - The documentation of G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START is confusing
The documentation of G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START is confusing
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gdbus
2.46.x
Other All
: Normal normal
: ---
Assigned To: David Zeuthen (not reading bugmail)
gtkdev
Depends on:
Blocks:
 
 
Reported: 2015-10-08 16:48 UTC by Debarshi Ray
Modified: 2015-10-09 12:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
docs: Improve the text on G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START (2.29 KB, patch)
2015-10-08 16:51 UTC, Debarshi Ray
none Details | Review
docs: Improve the text on G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START (2.29 KB, patch)
2015-10-09 09:52 UTC, Debarshi Ray
committed Details | Review

Description Debarshi Ray 2015-10-08 16:48:01 UTC
The inconsistent use of negations makes it hard to read, and it isn't clear how it is different from G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START_AT_CONSTRUCTION.
Comment 1 Debarshi Ray 2015-10-08 16:51:36 UTC
Created attachment 312916 [details] [review]
docs: Improve the text on G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START

Thanks to Jasper and Alex for clearing it up for me.
Comment 2 Dan Winship 2015-10-08 17:06:30 UTC
Comment on attachment 312916 [details] [review]
docs: Improve the text on G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START

>- * @G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START: If not set and the proxy if for a well-known name,
>- * then request the bus to launch an owner for the name if no-one owns the name. This flag can
>- * only be used in proxies for well-known names.
>+ * @G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START: If the proxy is for a well-known name,
>+ * do not ask the bus to launch an owner during proxy initialization or a method call.
>+ * This flag can only be used in proxies for well-known names.

"can only be used" isn't right. It's just that it has no effect otherwise. The DO_NOT_AUTO_START_AT_CONSTRUCTION docs phrase it as "This flag is only meaningful in proxies for well-known names", which seems like better wording.

other than that, looks good
Comment 3 Debarshi Ray 2015-10-09 09:52:50 UTC
Created attachment 312943 [details] [review]
docs: Improve the text on G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START
Comment 4 Debarshi Ray 2015-10-09 12:17:04 UTC
Comment on attachment 312943 [details] [review]
docs: Improve the text on G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START

Thanks, Dan. Pushed to master and glib-2-46.