Misc: Remove unnecessary \n from log messages

The final end-of-line marker is added in usbi_log_v() using the
USBI_LOG_LINE_END define.

Also don't mix hardcoded \n and USBI_LOG_LINE_END in usbi_log_v().

Closes #59

Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>

diff --git a/libusb/core.c b/libusb/core.c
index be658b0de6b535f82947f1d47b962dc7a3c110c3..65384ba37ae987a5c2e59cfdec76ec2d6e30d8ae 100644 (file)
--- a/libusb/core.c
+++ b/libusb/core.c
@@ -2309,8 +2309,8 @@ void usbi_log_v(struct libusb_context *ctx, enum libusb_log_level level,
        usbi_gettimeofday(&now, NULL);
        if ((global_debug) && (!has_debug_header_been_displayed)) {
                has_debug_header_been_displayed = 1;
-               usbi_log_str(ctx, LIBUSB_LOG_LEVEL_DEBUG, "[timestamp] [threadID] facility level [function call] <message>\n");
-               usbi_log_str(ctx, LIBUSB_LOG_LEVEL_DEBUG, "--------------------------------------------------------------------------------\n");
+               usbi_log_str(ctx, LIBUSB_LOG_LEVEL_DEBUG, "[timestamp] [threadID] facility level [function call] <message>" USBI_LOG_LINE_END);
+               usbi_log_str(ctx, LIBUSB_LOG_LEVEL_DEBUG, "--------------------------------------------------------------------------------" USBI_LOG_LINE_END);
        }
        if (now.tv_usec < timestamp_origin.tv_usec) {
                now.tv_sec--;

diff --git a/libusb/descriptor.c b/libusb/descriptor.c
index 53405e4955eef9937e5ffd64385704e8268e23a6..b9046577b573ce5cce8154824cc8266a539acc64 100644 (file)
--- a/libusb/descriptor.c
+++ b/libusb/descriptor.c
@@ -455,7 +455,7 @@ static int parse_configuration(struct libusb_context *ctx,
                                        (header.bDescriptorType == LIBUSB_DT_DEVICE))
                                break;
 
-                       usbi_dbg("skipping descriptor 0x%x\n", header.bDescriptorType);
+                       usbi_dbg("skipping descriptor 0x%x", header.bDescriptorType);
                        buffer += header.bLength;
                        size -= header.bLength;
                }

diff --git a/libusb/io.c b/libusb/io.c
index f85a209e82e6f23b95af414fc55387aa2b3e45af..0984099012657807052c194cd19777327641c65f 100644 (file)
--- a/libusb/io.c
+++ b/libusb/io.c
@@ -2109,7 +2109,7 @@ redo_poll:
        else if (r == -1 && errno == EINTR)
                return LIBUSB_ERROR_INTERRUPTED;
        else if (r < 0) {
-               usbi_err(ctx, "poll failed %d err=%d\n", r, errno);
+               usbi_err(ctx, "poll failed %d err=%d", r, errno);
                return LIBUSB_ERROR_IO;
        }
 

diff --git a/libusb/os/darwin_usb.c b/libusb/os/darwin_usb.c
index 9075de17cbfc255c4e9b4a09edee7ab5530605d2..e5d68aa87a7d9a8fe141f2c295a6503781900d5e 100644 (file)
--- a/libusb/os/darwin_usb.c
+++ b/libusb/os/darwin_usb.c
@@ -246,7 +246,7 @@ static usb_device_t **darwin_device_from_service (io_service_t service)
                                              &score);
 
   if (kIOReturnSuccess != result || !plugInInterface) {
-    usbi_dbg ("could not set up plugin for service: %s\n", darwin_error_str (result));
+    usbi_dbg ("could not set up plugin for service: %s", darwin_error_str (result));
     return NULL;
   }
 
@@ -998,7 +998,7 @@ static void darwin_close (struct libusb_device_handle *dev_handle) {
 
   if (dpriv->open_count == 0) {
     /* something is probably very wrong if this is the case */
-    usbi_err (HANDLE_CTX (dev_handle), "Close called on a device that was not open!\n");
+    usbi_err (HANDLE_CTX (dev_handle), "Close called on a device that was not open!");
     return;
   }
 

diff --git a/libusb/os/linux_usbfs.c b/libusb/os/linux_usbfs.c
index 5e9b6669594982013235cdc564f7df6e94f97b76..3879f2adf9310a1824c4a41f1a8afc5f9621b631 100644 (file)
--- a/libusb/os/linux_usbfs.c
+++ b/libusb/os/linux_usbfs.c
@@ -198,7 +198,7 @@ static int _get_usbfs_fd(struct libusb_device *dev, mode_t mode, int silent)
 
        if (errno == ENOENT) {
                if (!silent) 
-                       usbi_err(ctx, "File doesn't exist, wait %d ms and try again\n", delay/1000);
+                       usbi_err(ctx, "File doesn't exist, wait %d ms and try again", delay/1000);
    
                /* Wait 10ms for USB device path creation.*/
                usleep(delay);
@@ -2583,7 +2583,7 @@ static int op_handle_events(struct libusb_context *ctx,
                }
 
                if (!hpriv || hpriv->fd != pollfd->fd) {
-                       usbi_err(ctx, "cannot find handle for fd %d\n",
+                       usbi_err(ctx, "cannot find handle for fd %d",
                                 pollfd->fd);
                        continue;
                }

diff --git a/libusb/os/poll_windows.c b/libusb/os/poll_windows.c
index 593db5bcebebbaee563f91f782e58b3d2e34e6a3..822bb9409784a29c2dd742e6dde4b210f83912a1 100644 (file)
--- a/libusb/os/poll_windows.c
+++ b/libusb/os/poll_windows.c
@@ -333,7 +333,7 @@ struct winfd usbi_create_fd(HANDLE handle, int access_mode, struct usbi_transfer
        wfd.cancel_fn = cancel_fn;
 
        if ((access_mode != RW_READ) && (access_mode != RW_WRITE)) {
-               usbi_warn(NULL, "only one of RW_READ or RW_WRITE are supported.\n"
+               usbi_warn(NULL, "only one of RW_READ or RW_WRITE are supported. "
                        "If you want to poll for R/W simultaneously, create multiple fds from the same handle.");
                return INVALID_WINFD;
        }

diff --git a/libusb/os/windows_usb.c b/libusb/os/windows_usb.c
index 3c512f09cc5a5e0d1c11ab181de82de8ca57aef2..da9000d964f35c956f051c238208fb23af7f5757 100644 (file)
--- a/libusb/os/windows_usb.c
+++ b/libusb/os/windows_usb.c
@@ -1559,7 +1559,7 @@ static int windows_get_device_list(struct libusb_context *ctx, struct discovered
 //#define ENUM_DEBUG
 #ifdef ENUM_DEBUG
                const char *passname[] = { "HCD", "HUB", "GEN", "DEV", "HID", "EXT" };
-               usbi_dbg("\n#### PROCESSING %ss %s", passname[(pass<=HID_PASS)?pass:HID_PASS+1],
+               usbi_dbg("#### PROCESSING %ss %s", passname[(pass<=HID_PASS)?pass:HID_PASS+1],
                        (pass!=GEN_PASS)?guid_to_string(guid[pass]):"");
 #endif
                for (i = 0; ; i++) {
@@ -2678,7 +2678,7 @@ static int winusbx_init(int sub_api, struct libusb_context *ctx)
                if (h == NULL) {
                        h = LoadLibraryA("WinUSB");
                }               if (h == NULL) {
-                       usbi_warn(ctx, "WinUSB DLL is not available either,\n"
+                       usbi_warn(ctx, "WinUSB DLL is not available either, "
                                "you will not be able to access devices outside of enumeration");
                        return LIBUSB_ERROR_NOT_FOUND;
                }

diff --git a/libusb/version_nano.h b/libusb/version_nano.h
index a279ab1667e860fbe6a0e53d58f4d65cfc434da3..f5f4b29ef24497ee5def0737f61c67eaca4e75dd 100644 (file)
--- a/libusb/version_nano.h
+++ b/libusb/version_nano.h
@@ -1 +1 @@
-#define LIBUSB_NANO 10977
+#define LIBUSB_NANO 10978