Applied with commit msg modifications, Thanks On Fri, 2019-11-22 at 08:23 -0800, Inga Stotland wrote: > This removes an extra return parameter from PromptNumeric and PromptStatic > method signatures. Also, style fixes. > --- > tools/mesh-cfgclient.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/tools/mesh-cfgclient.c b/tools/mesh-cfgclient.c > index 1c617a37b..200eb5b84 100644 > --- a/tools/mesh-cfgclient.c > +++ b/tools/mesh-cfgclient.c > @@ -478,6 +478,7 @@ static struct l_dbus_message *disp_string_call(struct l_dbus *dbus, > struct l_dbus_message *msg, > void *user_data) > { > + const char *prompt = "Enter AlphaNumeric code on remote device:"; > char *str; > > if (!l_dbus_message_get_arguments(msg, "s", &str)) { > @@ -485,7 +486,7 @@ static struct l_dbus_message *disp_string_call(struct l_dbus *dbus, > return l_dbus_message_new_error(msg, dbus_err_fail, NULL); > } > > - bt_shell_printf(COLOR_YELLOW "Enter AlphaNumeric code on remote device: %s\n" COLOR_OFF, str); > + bt_shell_printf(COLOR_YELLOW "%s %s\n" COLOR_OFF, prompt, str); > > return l_dbus_message_new_method_return(msg); > } > @@ -542,13 +543,13 @@ static void setup_agent_iface(struct l_dbus_interface *iface) > NULL); > /* TODO: Other properties */ > l_dbus_interface_method(iface, "DisplayString", 0, disp_string_call, > - "", "s", "value"); > + "", "s", "value"); > l_dbus_interface_method(iface, "DisplayNumeric", 0, disp_numeric_call, > "", "su", "type", "number"); > l_dbus_interface_method(iface, "PromptNumeric", 0, prompt_numeric_call, > - "u", "s", "type"); > + "u", "s", "number", "type"); > l_dbus_interface_method(iface, "PromptStatic", 0, prompt_static_call, > - "ay", "s", "type"); > + "ay", "s", "data", "type"); > } > > static bool register_agent(void)