Apply suggestions from code review

Co-authored-by: overlookmotel <[email protected]>
Signed-off-by: Dunqing <[email protected]>

Author: Dunqing

crates/oxc_ast/src/ast/comment.rs

@@ -18,6 +18,7 @@ pub enum CommentKind {
     /// Block comment
     Block = 1,
     /// Multiline block comment (contains line breaks)
+    #[estree(rename = "Block")]
     MultilineBlock = 2,
 }
 

crates/oxc_parser/src/lexer/comment.rs

@@ -83,10 +83,13 @@ impl<'a> Lexer<'a> {
     /// Section 12.4 Multi Line Comment
     pub(super) fn skip_multi_line_comment(&mut self) -> Kind {
         // If `is_on_new_line` is already set, go directly to faster search which only looks for `*/`
-        // TODO(FIXME): commented out since it doesn't means there is a line break in the comment, while it means the line break is before the comment
-        // if self.token.is_on_new_line() {
-        //     return self.skip_multi_line_comment_after_line_break(self.source.position());
-        // }
+        // We need to identify if comment contains line breaks or not
+        // (`CommentKind::Block` or `CommentKind::MultilineBlock`).
+        // So we have to use the loop below for the first line of the comment even if
+        // `Token`'s `is_on_new_line` flag is already set.
+        // If the loop finds a line break before end of the comment, we then switch to
+        // the faster `skip_multi_line_comment_after_line_break` which searches 
+        // for the end of the comment using `memchr`.
 
         byte_search! {
             lexer: self,