Single line comments can end in newline or EOF. Fixes opensearch-proj…

…ect#11815 (opensearch-project#11816) * Single line comments can end in newline or EOF. Fixes opensearch-project#11815 Signed-off-by: camerondurham <[email protected]> * Re-add license to autogen parser files Signed-off-by: camerondurham <[email protected]> * Remove noise from changelog Signed-off-by: Cameron Durham <[email protected]> * Accept single-line comments until newline char Signed-off-by: Cameron Durham <[email protected]> --------- Signed-off-by: camerondurham <[email protected]> Signed-off-by: Cameron Durham <[email protected]>
peternied · Jan 10, 2024 · 8192488 · 8192488
1 parent 439573b
commit 8192488
Show file tree

Hide file tree

Showing 4 changed files with 427 additions and 375 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -218,6 +218,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 - Fix Automatic addition of protocol broken in #11512 ([#11609](https://github.com/opensearch-project/OpenSearch/pull/11609))
 - Fix issue when calling Delete PIT endpoint and no PITs exist ([#11711](https://github.com/opensearch-project/OpenSearch/pull/11711))
 - Fix tracing context propagation for local transport instrumentation ([#11490](https://github.com/opensearch-project/OpenSearch/pull/11490))
+- Fix parsing of single line comments in `lang-painless` ([#11815](https://github.com/opensearch-project/OpenSearch/issues/11815))
 
 ### Security
 

diff --git a/modules/lang-painless/src/main/antlr/PainlessLexer.g4 b/modules/lang-painless/src/main/antlr/PainlessLexer.g4
@@ -25,7 +25,7 @@ protected abstract boolean isSlashRegex();
 }
 
 WS: [ \t\n\r]+ -> skip;
-COMMENT: ( '//' .*? [\n\r] | '/*' .*? '*/' ) -> skip;
+COMMENT: ( '//' ~[\n\r]* | '/*' .*? '*/' ) -> skip;
 
 LBRACK:    '{';
 RBRACK:    '}';