fix: handle escaping of pipes as well as backslash itself p-e-w#65

Author: Luke Hudson

argos@pew.worldwidemann.com/utilities.js

@@ -69,22 +69,36 @@ export function parseFilename(filename) {
   return settings;
 }
 
+function locatePipe(str) {
+  let isEscaped = false;
+
+  for (let i = 0; i < str.length; i++) {
+    if (isEscaped) {
+      // Previous character was a backslash, reset state after current escaped character
+      isEscaped = false;
+      continue;
+    }
+
+    if (str[i] === "\\") {
+      // Enter escape state
+      isEscaped = true;
+      continue;
+    }
+
+    if (str[i] === "|" && !isEscaped) {
+      return i;
+    }
+  }
+
+  return -1;
+}
+
 // Performs (mostly) BitBar-compatible output line parsing
 // (see https://github.com/matryer/bitbar#plugin-api)
 export function parseLine(lineString) {
   let line = {};
-  let separatorIndex = undefined;
-
-  do
-  {
-    separatorIndex = lineString.indexOf("|", separatorIndex);
-    if (separatorIndex === 0 || lineString[separatorIndex-1] != '\\') {
-      // found a non-escaped pipe symbol
-      break;
-    }
-    // keep looking
-    ++separatorIndex;
-  } while(separatorIndex < lineString.length);
+
+  let separatorIndex = locatePipe(lineString);
 
   if (separatorIndex >= 0) {
     let attributes = [];