Hi Grzegorz,
On Thu, Apr 19, 2018 at 5:03 PM, Grzegorz Kolodziejczyk
<grzegorz.kolodziejczyk@xxxxxxxxxxx> wrote:
> This patch adds missing newline characters to shell printfs. It fixes
> printig issues.
We might be able to solve this sort of problem by introducing a macro,
e.g. bt_shell_msg, that adds the new line, for bt_shell_printf Id like
to keep as it is since there could be tools that need to break the
messages.
> ---
> client/gatt.c | 20 +++++++++++---------
> 1 file changed, 11 insertions(+), 9 deletions(-)
>
> diff --git a/client/gatt.c b/client/gatt.c
> index 7103c4f83..52a999dc9 100644
> --- a/client/gatt.c
> +++ b/client/gatt.c
> @@ -627,7 +627,8 @@ static void write_attribute(GDBusProxy *proxy, char *arg)
> bt_shell_printf("Attempting to write fd %d\n",
> io_get_fd(write_io.io));
> if (io_send(write_io.io, &iov, 1) < 0) {
> - bt_shell_printf("Failed to write: %s", strerror(errno));
> + bt_shell_printf("Failed to write: %s\n",
> + strerror(errno));
> return bt_shell_noninteractive_quit(EXIT_FAILURE);
> }
> return;
> @@ -1622,7 +1623,7 @@ static void authorize_write_response(const char *input, void *user_data)
> goto error;
> }
>
> - bt_shell_printf("[" COLORED_CHG "] Attribute %s written" , chrc->path);
> + bt_shell_printf("[" COLORED_CHG "] Attribute %s written\n", chrc->path);
>
> g_dbus_emit_property_changed(aad->conn, chrc->path, CHRC_INTERFACE,
> "Value");
> @@ -1671,7 +1672,7 @@ static DBusMessage *chrc_write_value(DBusConnection *conn, DBusMessage *msg,
> "org.bluez.Error.InvalidArguments",
> NULL);
>
> - bt_shell_printf("[" COLORED_CHG "] Attribute %s written" , chrc->path);
> + bt_shell_printf("[" COLORED_CHG "] Attribute %s written\n", chrc->path);
>
> g_dbus_emit_property_changed(conn, chrc->path, CHRC_INTERFACE, "Value");
>
> @@ -1789,8 +1790,8 @@ static DBusMessage *chrc_start_notify(DBusConnection *conn, DBusMessage *msg,
> return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
>
> chrc->notifying = true;
> - bt_shell_printf("[" COLORED_CHG "] Attribute %s notifications enabled",
> - chrc->path);
> + bt_shell_printf("[" COLORED_CHG "] Attribute %s notifications "
> + "enabled\n", chrc->path);
> g_dbus_emit_property_changed(conn, chrc->path, CHRC_INTERFACE,
> "Notifying");
>
> @@ -1806,8 +1807,8 @@ static DBusMessage *chrc_stop_notify(DBusConnection *conn, DBusMessage *msg,
> return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
>
> chrc->notifying = false;
> - bt_shell_printf("[" COLORED_CHG "] Attribute %s notifications disabled",
> - chrc->path);
> + bt_shell_printf("[" COLORED_CHG "] Attribute %s notifications "
> + "disabled\n", chrc->path);
> g_dbus_emit_property_changed(conn, chrc->path, CHRC_INTERFACE,
> "Notifying");
>
> @@ -1819,7 +1820,8 @@ static DBusMessage *chrc_confirm(DBusConnection *conn, DBusMessage *msg,
> {
> struct chrc *chrc = user_data;
>
> - bt_shell_printf("Attribute %s indication confirm received", chrc->path);
> + bt_shell_printf("Attribute %s indication confirm received\n",
> + chrc->path);
>
> return dbus_message_new_method_return(msg);
> }
> @@ -2012,7 +2014,7 @@ static DBusMessage *desc_write_value(DBusConnection *conn, DBusMessage *msg,
> bt_shell_printf("WriteValue: %s offset %u link %s\n",
> path_to_address(device), offset, link);
>
> - bt_shell_printf("[" COLORED_CHG "] Attribute %s written" , desc->path);
> + bt_shell_printf("[" COLORED_CHG "] Attribute %s written\n" , desc->path);
>
> g_dbus_emit_property_changed(conn, desc->path, CHRC_INTERFACE, "Value");
>
> --
> 2.13.6
>
> --
> 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
--
Luiz Augusto von Dentz
--
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
