Diff between 80cee3a56c2066737c6915772c034e4ad3a1de32 and 5c3a39e642d5e499ade0a2cdf400a9ccd7804a19

Changed Files

File Additions Deletions Status
client/advertising.c +27 -1 modified
client/advertising.h +1 -0 modified
client/main.c +10 -1 modified

Full Patch

diff --git a/client/advertising.c b/client/advertising.c
index 571b6e6..696621b 100644
--- a/client/advertising.c
+++ b/client/advertising.c
@@ -58,6 +58,7 @@ static struct ad {
 	bool registered;
 	char *type;
 	char *local_name;
+	uint16_t local_appearance;
 	char **uuids;
 	size_t uuids_len;
 	struct service_data service;
@@ -65,7 +66,9 @@ static struct ad {
 	bool tx_power;
 	bool name;
 	bool appearance;
-} ad;
+} ad = {
+	.local_appearance = UINT16_MAX,
+};
 
 static void ad_release(DBusConnection *conn)
 {
@@ -304,6 +307,20 @@ static gboolean get_local_name(const GDBusPropertyTable *property,
 	return TRUE;
 }
 
+static gboolean appearance_exits(const GDBusPropertyTable *property, void *data)
+{
+	return ad.local_appearance != UINT16_MAX ? TRUE : FALSE;
+}
+
+static gboolean get_appearance(const GDBusPropertyTable *property,
+				DBusMessageIter *iter, void *user_data)
+{
+	dbus_message_iter_append_basic(iter, DBUS_TYPE_UINT16,
+							&ad.local_appearance);
+
+	return TRUE;
+}
+
 static const GDBusPropertyTable ad_props[] = {
 	{ "Type", "s", get_type },
 	{ "ServiceUUIDs", "as", get_uuids, NULL, uuids_exists },
@@ -312,6 +329,7 @@ static const GDBusPropertyTable ad_props[] = {
 						manufacturer_data_exists },
 	{ "Includes", "as", get_includes, NULL, includes_exists },
 	{ "LocalName", "s", get_local_name, NULL, local_name_exits },
+	{ "Appearance", "q", get_appearance, NULL, appearance_exits },
 	{ }
 };
 
@@ -524,4 +542,12 @@ void ad_advertise_local_name(const char *name)
 void ad_advertise_appearance(bool value)
 {
 	ad.appearance = value;
+
+	if (!value)
+		ad.local_appearance = UINT16_MAX;
+}
+
+void ad_advertise_local_appearance(uint16_t value)
+{
+	ad.local_appearance = value;
 }
diff --git a/client/advertising.h b/client/advertising.h
index 2557435..5ea57ad 100644
--- a/client/advertising.h
+++ b/client/advertising.h
@@ -31,3 +31,4 @@ void ad_advertise_tx_power(bool value);
 void ad_advertise_name(bool value);
 void ad_advertise_appearance(bool value);
 void ad_advertise_local_name(const char *name);
+void ad_advertise_local_appearance(uint16_t value);
diff --git a/client/main.c b/client/main.c
index 328a5ee..6b1c73f 100644
--- a/client/main.c
+++ b/client/main.c
@@ -2384,6 +2384,9 @@ static void cmd_set_advertise_name(const char *arg)
 
 static void cmd_set_advertise_appearance(const char *arg)
 {
+	long int value;
+	char *endptr = NULL;
+
 	if (arg == NULL || strlen(arg) == 0) {
 		rl_printf("Missing value argument\n");
 		return;
@@ -2399,7 +2402,13 @@ static void cmd_set_advertise_appearance(const char *arg)
 		return;
 	}
 
-	rl_printf("Invalid argument\n");
+	value = strtol(arg, &endptr, 0);
+	if (!endptr || *endptr != '\0' || value > UINT16_MAX) {
+		rl_printf("Invalid argument\n");
+		return;
+	}
+
+	ad_advertise_local_appearance(value);
 }
 
 static const struct {