fix: make ast.errors empty (#21)

* fix: make ast.errors empty

* perf: improve memory efficiency of computeLineMap

src/parseWithPointers.ts

@@ -50,6 +50,10 @@ export const parseWithPointers = <T>(value: string, options?: IParseOptions): Ya
     parsed.diagnostics.sort((itemA, itemB) => itemA.range.start.line - itemB.range.start.line);
   }
 
+  if (Array.isArray(parsed.ast.errors)) {
+    parsed.ast.errors.length = 0;
+  }
+
   return parsed;
 };
 
@@ -148,15 +152,17 @@ const dereferenceAnchor = (node: YAMLNode, anchorId: string): YAMLNode | YAMLNod
 
 // builds up the line map, for use by linesForPosition
 const computeLineMap = (input: string) => {
-  const lines = input.split(/\n/);
   const lineMap: number[] = [];
 
-  let sum = 0;
-  for (const line of lines) {
-    sum += line.length + 1;
-    lineMap.push(sum);
+  let i = 0;
+  for (; i < input.length; i++) {
+    if (input[i] === '\n') {
+      lineMap.push(i + 1);
+    }
   }
 
+  lineMap.push(i + 1);
+
   return lineMap;
 };