From: Marcel Holtmann <marcel@holtmann.org>
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH BlueZ 06/11] gdbus: Introduce G_DBUS_FLAG_ENABLE_DEPRECATED
Date: Sun, 30 Dec 2012 10:52:51 -0800 [thread overview]
Message-ID: <1356893571.19248.84.camel@aeonflux> (raw)
In-Reply-To: <1356699069-5469-7-git-send-email-luiz.dentz@gmail.com>
Hi Luiz,
> This flag can be set using g_dbus_set_flags to enable deprecated
> interfaces, default is unset.
> ---
> gdbus/gdbus.h | 3 ++-
> gdbus/object.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 46 insertions(+), 1 deletion(-)
I decided against this patch. At least for now. The whole point of the
interface versioning is to support two API versions. So having an option
to manually enable them makes no sense. We can revisit this once we
reach that point, but otherwise I leave this out for now.
Regards
Marcel
next prev parent reply other threads:[~2012-12-30 18:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-28 12:50 [PATCH BlueZ 00/11] Add experimental command line switch option Luiz Augusto von Dentz
2012-12-28 12:50 ` [PATCH BlueZ 01/11] gdbus: Introduce G_DBUS_METHOD_FLAG_EXPERIMENTAL Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 02/11] gdbus: Introduce G_DBUS_SIGNAL_FLAG_EXPERIMENTAL Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 03/11] gdbus: Introduce G_DBUS_PROPERTY_FLAG_EXPERIMENTAL Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 04/11] gdbus: Check if the interface being registered is valid Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 05/11] gdbus: Call check_signals when sending signals with g_dbus_send_message Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 06/11] gdbus: Introduce G_DBUS_FLAG_ENABLE_DEPRECATED Luiz Augusto von Dentz
2012-12-30 18:52 ` Marcel Holtmann [this message]
2012-12-28 12:51 ` [PATCH BlueZ 07/11] core: Add command line switch for enabling experimental interfaces Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 08/11] core: Reuse --compat/-C switch for enabling deprecated interfaces Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 09/11] media: Enable RegisterPlayer and UnregisterPlayer methods as experimental Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 10/11] player: Enable MediaPlayer1 interface " Luiz Augusto von Dentz
2012-12-28 12:51 ` [PATCH BlueZ 11/11] AVRCP: Fix not checking for media_player_controller_create Luiz Augusto von Dentz
2012-12-30 19:02 ` [PATCH BlueZ 00/11] Add experimental command line switch option Marcel Holtmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1356893571.19248.84.camel@aeonflux \
--to=marcel@holtmann.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).