Re: [PATCH BlueZ 3/3] client: Always start an agent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

> Always register agent with 'KeyboardDisplay' capability.
> ---
> client/main.c | 18 ++++--------------
> 1 file changed, 4 insertions(+), 14 deletions(-)
> 
> diff --git a/client/main.c b/client/main.c
> index 8c9d5c3..7043081 100644
> --- a/client/main.c
> +++ b/client/main.c
> @@ -54,6 +54,8 @@
> #define PROMPT_ON	COLOR_BLUE "[bluetooth]" COLOR_OFF "# "
> #define PROMPT_OFF	"Waiting to connect to bluetoothd..."
> 
> +#define AGENT_CAP	"KeyboardDisplay"
> +
> static GMainLoop *main_loop;
> static DBusConnection *dbus_conn;
> 
> @@ -2352,23 +2354,9 @@ static guint setup_signalfd(void)
> 
> static gboolean option_version = FALSE;
> 
> -static gboolean parse_agent(const char *key, const char *value,
> -					gpointer user_data, GError **error)
> -{
> -	if (value)
> -		auto_register_agent = g_strdup(value);
> -	else
> -		auto_register_agent = g_strdup("");
> -
> -	return TRUE;
> -}
> -
> static GOptionEntry options[] = {
> 	{ "version", 'v', 0, G_OPTION_ARG_NONE, &option_version,
> 				"Show version information and exit" },
> -	{ "agent", 'a', G_OPTION_FLAG_OPTIONAL_ARG,
> -				G_OPTION_ARG_CALLBACK, parse_agent,
> -				"Register agent handler", "CAPABILITY" },
> 	{ NULL },
> };

actually leave the agent option and make the argument mandatory. We should be registering with “” as agent string by default. That ensures that we use the defaults from bluetoothd and mgmt. However the command line option can overwrite the IO capability.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux