add comments

ben-schwen · ben-schwen · commit cc57d2798f39 · 2025-10-18T23:59:38.000+02:00
diff --git a/src/fread.c b/src/fread.c
@@ -2309,6 +2309,7 @@ int freadMain(freadMainArgs _args)
           ch = skip_to_eol(commentPos, eof);
         }
       }
+      // consider different cases line ending after column names
       if (ch == eof || *ch == '\0') {
         pos = ch;
       } else if (*ch == '\n' || *ch == '\r') {
@@ -2318,7 +2319,7 @@ int freadMain(freadMainArgs _args)
         } else {
           INTERNAL_STOP("reading colnames ending on '%c'", *ch); // # nocov
         }
-      } else if (ch > sof && (ch[-1] == '\n' || ch[-1] == '\r')) {
+      } else if (ch > sof && (ch[-1] == '\n' || ch[-1] == '\r')) { // trimmed a comment and now on next rows first byte
         pos = ch;
       } else {
         INTERNAL_STOP("reading colnames ending on '%c'", *ch); // # nocov

Original file line number	Diff line number	Diff line change
`@@ -2309,6 +2309,7 @@ int freadMain(freadMainArgs _args)`
`2309`	`2309`	`ch = skip_to_eol(commentPos, eof);`
`2310`	`2310`	`}`
`2311`	`2311`	`}`
	`2312`	`+ // consider different cases line ending after column names`
`2312`	`2313`	`if (ch == eof \|\| *ch == '\0') {`
`2313`	`2314`	`pos = ch;`
`2314`	`2315`	`} else if (ch == '\n' \|\| ch == '\r') {`
`@@ -2318,7 +2319,7 @@ int freadMain(freadMainArgs _args)`
`2318`	`2319`	`} else {`
`2319`	`2320`	`INTERNAL_STOP("reading colnames ending on '%c'", *ch); // # nocov`
`2320`	`2321`	`}`
`2321`		`- } else if (ch > sof && (ch[-1] == '\n' \|\| ch[-1] == '\r')) {`
	`2322`	`+ } else if (ch > sof && (ch[-1] == '\n' \|\| ch[-1] == '\r')) { // trimmed a comment and now on next rows first byte`
`2322`	`2323`	`pos = ch;`
`2323`	`2324`	`} else {`
`2324`	`2325`	`INTERNAL_STOP("reading colnames ending on '%c'", *ch); // # nocov`