all: fix some comments

Change-Id: Id334943c9be6e035d52a8d8a4bd26e0e31ae9791
Reviewed-on: https://go-review.googlesource.com/c/tools/+/492578
Reviewed-by: Robert Findley <[email protected]>
Reviewed-by: Tim King <[email protected]>
Run-TryBot: shuang cui <[email protected]>
Run-TryBot: Robert Findley <[email protected]>
TryBot-Result: Gopher Robot <[email protected]>
gopls-CI: kokoro <[email protected]>

Author: cuishuang

cmd/guru/guru.go

@@ -337,7 +337,7 @@ func deref(typ types.Type) types.Type {
 //   - a QueryPos, denoting the extent of the user's query.
 //   - nil, meaning no position at all.
 //
-// The output format is is compatible with the 'gnu'
+// The output format is compatible with the 'gnu'
 // compilation-error-regexp in Emacs' compilation mode.
 func fprintf(w io.Writer, fset *token.FileSet, pos interface{}, format string, args ...interface{}) {
 	var start, end token.Pos

cmd/present/static/notes.js

@@ -26,7 +26,7 @@ function toggleNotesWindow() {
   initNotes();
 }
 
-// Create an unique key for the local storage so we don't mix the
+// Create a unique key for the local storage so we don't mix the
 // destSlide of different presentations. For golang.org/issue/24688.
 function destSlideKey() {
   var key = '';

go/analysis/passes/ifaceassert/parameterized.go

@@ -67,7 +67,7 @@ func (w *tpWalker) isParameterized(typ types.Type) (res bool) {
 		// of a generic function type (or an interface method) that is
 		// part of the type we're testing. We don't care about these type
 		// parameters.
-		// Similarly, the receiver of a method may declare (rather then
+		// Similarly, the receiver of a method may declare (rather than
 		// use) type parameters, we don't care about those either.
 		// Thus, we only need to look at the input and result parameters.
 		return w.isParameterized(t.Params()) || w.isParameterized(t.Results())

go/analysis/passes/printf/printf.go

@@ -278,7 +278,7 @@ func checkPrintfFwd(pass *analysis.Pass, w *printfWrapper, call *ast.CallExpr, k
 			// print/printf function can take, adding an ellipsis
 			// would break the program. For example:
 			//
-			//   func foo(arg1 string, arg2 ...interface{} {
+			//   func foo(arg1 string, arg2 ...interface{}) {
 			//       fmt.Printf("%s %v", arg1, arg2)
 			//   }
 			return

go/ssa/parameterized.go

@@ -63,7 +63,7 @@ func (w *tpWalker) isParameterized(typ types.Type) (res bool) {
 		// of a generic function type (or an interface method) that is
 		// part of the type we're testing. We don't care about these type
 		// parameters.
-		// Similarly, the receiver of a method may declare (rather then
+		// Similarly, the receiver of a method may declare (rather than
 		// use) type parameters, we don't care about those either.
 		// Thus, we only need to look at the input and result parameters.
 		return w.isParameterized(t.Params()) || w.isParameterized(t.Results())

godoc/dirtrees.go

@@ -220,7 +220,7 @@ func (c *Corpus) newDirectory(root string, maxDepth int) *Directory {
 	// The root could be a symbolic link so use Stat not Lstat.
 	d, err := c.fs.Stat(root)
 	// If we fail here, report detailed error messages; otherwise
-	// is is hard to see why a directory tree was not built.
+	// is hard to see why a directory tree was not built.
 	switch {
 	case err != nil:
 		log.Printf("newDirectory(%s): %s", root, err)

gopls/internal/lsp/cache/check.go

@@ -322,7 +322,7 @@ type (
 // forEachPackage does a pre- and post- order traversal of the packages
 // specified by ids using the provided pre and post functions.
 //
-// The pre func is is optional. If set, pre is evaluated after the package
+// The pre func is optional. If set, pre is evaluated after the package
 // handle has been constructed, but before type-checking. If pre returns false,
 // type-checking is skipped for this package handle.
 //