common: Fix LoggerTest on Windows

\r was making it in the stored messages. It can be ignored.

Change-Id: I1e79a876e8cc5b610b8bccee234648dbe16bbf19
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93761
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI

diff --git a/common/org.eclipse.tracecompass.common.core.tests/src/org/eclipse/tracecompass/common/core/tests/log/LoggerTest.java b/common/org.eclipse.tracecompass.common.core.tests/src/org/eclipse/tracecompass/common/core/tests/log/LoggerTest.java
index cfc2c43a4c4e4840217a020f263461dc00d6a282..1ac78d2c7e3e33d30a7d67ebf0b56cf4f2d0afdc 100644 (file)
--- a/common/org.eclipse.tracecompass.common.core.tests/src/org/eclipse/tracecompass/common/core/tests/log/LoggerTest.java
+++ b/common/org.eclipse.tracecompass.common.core.tests/src/org/eclipse/tracecompass/common/core/tests/log/LoggerTest.java
@@ -56,6 +56,12 @@ public class LoggerTest {
 
         @Override
         public void write(int b) throws IOException {
+            // We don't care about carriage return (Windows). We only need to
+            // rely on \n to detect the next line
+            if (b == '\r') {
+                return;
+            }
+
             if (b != '\n') {
                 if (secondLine) {
                     if (start) {