Till Westmann has submitted this change and it was merged. Change subject: Fix for ASTERIXDB-1471 ......................................................................
Fix for ASTERIXDB-1471 The line number is increased twice in setBuffer and updateLineColumn which causes the line number is 2x larger. Change-Id: Ibd91bc13c3adbbbe1dbfa0dccbdf6b378b280eae Reviewed-on: https://asterix-gerrit.ics.uci.edu/1012 Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Reviewed-by: Michael Blow <michael.b...@couchbase.com> --- M asterixdb/asterix-maven-plugins/lexer-generator-maven-plugin/src/main/resources/Lexer.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Michael Blow: Looks good to me, approved Jenkins: Verified; No violations found; Verified diff --git a/asterixdb/asterix-maven-plugins/lexer-generator-maven-plugin/src/main/resources/Lexer.java b/asterixdb/asterix-maven-plugins/lexer-generator-maven-plugin/src/main/resources/Lexer.java index 60e6e89..495a95e 100644 --- a/asterixdb/asterix-maven-plugins/lexer-generator-maven-plugin/src/main/resources/Lexer.java +++ b/asterixdb/asterix-maven-plugins/lexer-generator-maven-plugin/src/main/resources/Lexer.java @@ -85,7 +85,6 @@ this.buffer = buffer; tokenBegin = bufpos = 0; containsEscapes = false; - line++; tokenBegin = -1; } -- To view, visit https://asterix-gerrit.ics.uci.edu/1012 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd91bc13c3adbbbe1dbfa0dccbdf6b378b280eae Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Xikui Wang <xkk...@gmail.com> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Michael Blow <michael.b...@couchbase.com> Gerrit-Reviewer: Till Westmann <ti...@apache.org>