Merge pull request #18 from Matts966/hotfix/newline

Fix redundant newline

Author: Matts966

zetasql/parser/unparser.cc

@@ -50,16 +50,19 @@ std::string Unparse(const ASTNode* node) {
 
 std::string UnparseWithComments(const ASTNode* node, std::deque<std::pair<std::string,
                                 ParseLocationPoint>>& parse_tokens) {
-  std::string unparsed_;
-  parser::Unparser unparser(&unparsed_);
+  std::string unparsed;
+  parser::Unparser unparser(&unparsed);
   // Print comments by visitors and pop.
   node->Accept(&unparser, &parse_tokens);
   // Emit left comments in parse_tokens.
   for (const auto& parse_token : parse_tokens) {
     unparser.print(parse_token.first);
   }
   unparser.FlushLine();
-  return unparsed_;
+  while (unparsed.back() == '\n' && unparsed.at(unparsed.size() - 2) == '\n') {
+    unparsed.pop_back();
+  }
+  return unparsed;
 }
 
 namespace parser {

zetasql/tools/zetasql-formatter/example_tests_formatted/missing_comment.sql

@@ -8,4 +8,3 @@ FROM
   fuga
   ON hoge.hoge = fuga.fuga;
 -- comment3
-