Commit 5e1b5f03 authored by Jakub Adam's avatar Jakub Adam Committed by Philip Withnall

agent: Improve debug in component_io_cb()

agent_recv_message_unlocked() always receives a single message and
returns a RecvStatus code.

Avoid weird debugs like "received -1 valid messages" (when retval is
RECV_WOULD_BLOCK) and print the message only when something was actually
received.
Reviewed-by: default avatarPhilip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.freedesktop.org/D1706
parent 01453935
...@@ -4975,21 +4975,26 @@ component_io_cb (GSocket *gsocket, GIOCondition condition, gpointer user_data) ...@@ -4975,21 +4975,26 @@ component_io_cb (GSocket *gsocket, GIOCondition condition, gpointer user_data)
retval = agent_recv_message_unlocked (agent, stream, component, retval = agent_recv_message_unlocked (agent, stream, component,
socket_source->socket, &local_message); socket_source->socket, &local_message);
nice_debug_verbose ("%s: %p: received %d valid messages with %" G_GSSIZE_FORMAT
" bytes", G_STRFUNC, agent, retval, local_message.length);
if (retval == RECV_WOULD_BLOCK) { if (retval == RECV_WOULD_BLOCK) {
/* EWOULDBLOCK. */ /* EWOULDBLOCK. */
nice_debug_verbose ("%s: %p: no message available on read attempt",
G_STRFUNC, agent);
break; break;
} else if (retval == RECV_ERROR) { } else if (retval == RECV_ERROR) {
/* Other error. */ /* Other error. */
nice_debug ("%s: error receiving message", G_STRFUNC); nice_debug ("%s: %p: error receiving message", G_STRFUNC, agent);
remove_source = TRUE; remove_source = TRUE;
break; break;
} }
if (retval == RECV_SUCCESS && local_message.length > 0) if (retval == RECV_SUCCESS) {
nice_component_emit_io_callback (component, local_buf, local_message.length); nice_debug_verbose ("%s: %p: received a valid message with %" G_GSSIZE_FORMAT
" bytes", G_STRFUNC, agent, local_message.length);
if (local_message.length > 0) {
nice_component_emit_io_callback (component, local_buf, local_message.length);
}
}
if (g_source_is_destroyed (g_main_current_source ())) { if (g_source_is_destroyed (g_main_current_source ())) {
nice_debug ("Component IO source disappeared during the callback"); nice_debug ("Component IO source disappeared during the callback");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment