chore: fix some minor issues in comments (#4462)

Author: criciss

internal/js_lexer/js_lexer.go

@@ -1713,7 +1713,7 @@ func (lexer *Lexer) scanIdentifierWithEscapes(kind identifierKind) (MaybeSubstri
 	//   // This is an error (equivalent to "var foo;" except for this rule)
 	//   \u0076\u0061\u0072 foo;
 	//
-	//   // This is an fine (equivalent to "foo.var;")
+	//   // This is a fine (equivalent to "foo.var;")
 	//   foo.\u0076\u0061\u0072;
 	//
 	if Keywords[text] != 0 {

internal/js_parser/js_parser.go

@@ -18228,7 +18228,7 @@ const (
 	whyESMImportStatement
 )
 
-// Say why this the current file is being considered an ES module
+// Say why the current file is being considered an ES module
 func (p *parser) whyESModule() (whyESM, []logger.MsgData) {
 	because := "This file is considered to be an ECMAScript module because"
 	switch {

internal/js_printer/js_printer.go

@@ -1759,7 +1759,7 @@ func (p *printer) guardAgainstBehaviorChangeDueToSubstitution(expr js_ast.Expr,
 
 // Constant folding is already implemented once in the parser. A smaller form
 // of constant folding (just for numbers) is implemented here to clean up cross-
-// module numeric constants and bitwise operations. This is not an general-
+// module numeric constants and bitwise operations. This is not a general-
 // purpose/optimal approach and never will be. For example, we can't affect
 // tree shaking at this stage because it has already happened.
 func (p *printer) lateConstantFoldUnaryOrBinaryOrIfExpr(expr js_ast.Expr) js_ast.Expr {

internal/linker/linker.go

@@ -5330,12 +5330,12 @@ func (c *linkerContext) renameSymbolsInChunk(chunk *chunkInfo, filesInOrder []ui
 	reservedNames := renamer.ComputeReservedNames(moduleScopes, c.graph.Symbols)
 
 	// Node contains code that scans CommonJS modules in an attempt to statically
-	// detect the  set of export names that a module will use. However, it doesn't
+	// detect the set of export names that a module will use. However, it doesn't
 	// do any scope analysis so it can be fooled by local variables with the same
 	// name as the CommonJS module-scope variables "exports" and "module". Avoid
 	// using these names in this case even if there is not a risk of a name
 	// collision because there is still a risk of node incorrectly detecting
-	// something in a nested scope as an top-level export. Here's a case where
+	// something in a nested scope as a top-level export. Here's a case where
 	// this happened: https://github.com/evanw/esbuild/issues/3544
 	if c.options.OutputFormat == config.FormatCommonJS && c.options.Platform == config.PlatformNode {
 		reservedNames["exports"] = 1