Fix pgbench lexer's "continuation" rule to cope with Windows newlines.

Our general practice in frontend code is to accept input with either
Unix-style newlines (\n) or DOS-style (\r\n).  pgbench was mostly down
with that, but its rule for line continuations (backslash-newline) was
not.  This had been masked on Windows buildfarm machines before commit
0ba06e0bf by use of Windows text mode to read files.  We could have fixed
it by forcing text mode again, but it's better to fix the parsing code
so that Windows-style text files on Unix systems don't cause problems.

Back-patch to v10 where pgbench grew line continuations.

Discussion: https://postgr.es/m/17194.1537191697@sss.pgh.pa.us

diff --git a/src/bin/pgbench/exprscan.l b/src/bin/pgbench/exprscan.l
index 5c1bd881283d2f05e27deb10d9e64f822631a863..61c20364ed1c0643c51033a1b0d01a272f0fabe4 100644 (file)
--- a/src/bin/pgbench/exprscan.l
+++ b/src/bin/pgbench/exprscan.l
@@ -69,7 +69,7 @@ nonspace              [^ \t\r\f\v\n]
 newline                        [\n]
 
 /* Line continuation marker */
-continuation   \\{newline}
+continuation   \\\r?{newline}
 
 /* case insensitive keywords */
 and                            [Aa][Nn][Dd]
@@ -122,8 +122,12 @@ notnull                    [Nn][Oo][Tt][Nn][Uu][Ll][Ll]
         * a continuation marker just after a word:
         */
 {nonspace}+{continuation}      {
-                                       /* Found "word\\\n", emit and return just "word" */
-                                       psqlscan_emit(cur_state, yytext, yyleng - 2);
+                                       /* Found "word\\\r?\n", emit and return just "word" */
+                                       int             wordlen = yyleng - 2;
+                                       if (yytext[wordlen] == '\r')
+                                               wordlen--;
+                                       Assert(yytext[wordlen] == '\\');
+                                       psqlscan_emit(cur_state, yytext, wordlen);
                                        return 1;
                                }