commit b5b9500a62515b65725fa8f9b4b4428fd982e000
author Christopher N. Hesse <raymanfx@gmail.com> Sat Mar 04 15:04:16 2017 +0100
committer Christopher N. Hesse <raymanfx@gmail.com> Sat Mar 04 15:04:16 2017 +0100
tree 22da91a49e79ced90f9cbfb5ffd1e6454b0c548e
parent 354f713a56ec4de3f5db90b9175aa9884146430b

lights: Remove superfluous log newlines

Change-Id: Id1522dfe92c9e585000edd7067b50968f38c07ca

liblights/lights.c

diff --git a/liblights/lights.c b/liblights/lights.c
index f7d1d60..7af7702 100644
--- a/liblights/lights.c
+++ b/liblights/lights.c
@@ -99,7 +99,7 @@
         return amt == -1 ? -errno : 0;
     } else {
         if (already_warned == 0) {
-            ALOGE("write_str failed to open %s\n", path);
+            ALOGE("write_str failed to open %s", path);
             already_warned = 1;
         }
         return -errno;
@@ -128,7 +128,7 @@
     if (max_brightness > MAX_INPUT_BRIGHTNESS) {
         int old_brightness = brightness;
         brightness = brightness * max_brightness / MAX_INPUT_BRIGHTNESS;
-        ALOGV("%s: scaling brightness %d => %d\n", __func__,
+        ALOGV("%s: scaling brightness %d => %d", __func__,
             old_brightness, brightness);
     }
 
@@ -190,13 +190,9 @@
         return -EINVAL;
     }
 
-    ALOGV("%s: color=0x%08x, delay_on=%d, delay_off=%d, blink=\"%s\".",
+    ALOGV("%s: color=0x%08x, delay_on=%d, delay_off=%d, blink=%s",
           __func__, led->color, led->delay_on, led->delay_off, blink);
 
-    /* Add '\n' here to make the above log message clean. */
-    blink[count]   = '\n';
-    blink[count+1] = '\0';
-
     pthread_mutex_lock(&g_lock);
     err = write_str(LED_BLINK_NODE, blink);
     pthread_mutex_unlock(&g_lock);