OSDN Git Service

android/socket: Strip extra log messages
authorAndrei Emeltchenko <andrei.emeltchenko@intel.com>
Thu, 28 Nov 2013 14:38:04 +0000 (16:38 +0200)
committerJohan Hedberg <johan.hedberg@intel.com>
Fri, 29 Nov 2013 08:30:02 +0000 (10:30 +0200)
Remove debug messages when sending data, debug still exist for connection
establishment. Do not print error when connection hang up, print debug
instead.

android/socket.c

index 85f1ec4..6e3ef26 100644 (file)
@@ -488,11 +488,12 @@ static gboolean sock_stack_event_cb(GIOChannel *io, GIOCondition cond,
        unsigned char buf[1024];
        int len, sent;
 
-       DBG("rfsock: fd %d real_sock %d chan %u sock %d",
-               rfsock->fd, rfsock->real_sock, rfsock->channel,
-               g_io_channel_unix_get_fd(io));
+       if (cond & G_IO_HUP) {
+               DBG("Socket %d hang up", g_io_channel_unix_get_fd(io));
+               goto fail;
+       }
 
-       if (cond & (G_IO_ERR | G_IO_HUP | G_IO_NVAL)) {
+       if (cond & (G_IO_ERR | G_IO_NVAL)) {
                error("Socket error: sock %d cond %d",
                                        g_io_channel_unix_get_fd(io), cond);
                goto fail;
@@ -505,16 +506,12 @@ static gboolean sock_stack_event_cb(GIOChannel *io, GIOCondition cond,
                return TRUE;
        }
 
-       DBG("read %d bytes write to %d", len, rfsock->real_sock);
-
        sent = try_write_all(rfsock->real_sock, buf, len);
        if (sent < 0) {
                error("write(): %s", strerror(errno));
                goto fail;
        }
 
-       DBG("Written %d bytes", sent);
-
        return TRUE;
 fail:
        connections = g_list_remove(connections, rfsock);
@@ -530,11 +527,12 @@ static gboolean sock_rfcomm_event_cb(GIOChannel *io, GIOCondition cond,
        unsigned char buf[1024];
        int len, sent;
 
-       DBG("rfsock: fd %d real_sock %d chan %u sock %d",
-               rfsock->fd, rfsock->real_sock, rfsock->channel,
-               g_io_channel_unix_get_fd(io));
+       if (cond & G_IO_HUP) {
+               DBG("Socket %d hang up", g_io_channel_unix_get_fd(io));
+               goto fail;
+       }
 
-       if (cond & (G_IO_ERR | G_IO_HUP | G_IO_NVAL)) {
+       if (cond & (G_IO_ERR | G_IO_NVAL)) {
                error("Socket error: sock %d cond %d",
                                        g_io_channel_unix_get_fd(io), cond);
                goto fail;
@@ -547,16 +545,12 @@ static gboolean sock_rfcomm_event_cb(GIOChannel *io, GIOCondition cond,
                return TRUE;
        }
 
-       DBG("read %d bytes, write to fd %d", len, rfsock->fd);
-
        sent = try_write_all(rfsock->fd, buf, len);
        if (sent < 0) {
                error("write(): %s", strerror(errno));
                goto fail;
        }
 
-       DBG("Written %d bytes", sent);
-
        return TRUE;
 fail:
        connections = g_list_remove(connections, rfsock);