all: fix some minor issues in comments

Change-Id: I662505f503cc3e4a2e3fdd0f13b6596fd7388e56
Reviewed-on: https://go-review.googlesource.com/c/tools/+/688675
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Alan Donovan <adonovan@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Alan Donovan <adonovan@google.com>

Author: cuishuang

go/analysis/analysistest/analysistest.go

@@ -575,7 +575,7 @@ func check(t Testing, gopath string, act *checker.Action) {
 	files := act.Package.OtherFiles
 
 	// Hack: these two analyzers need to extract expectations from
-	// all configurations, so include the files are are usually
+	// all configurations, so include the files are usually
 	// ignored. (This was previously a hack in the respective
 	// analyzers' tests.)
 	if act.Analyzer.Name == "buildtag" || act.Analyzer.Name == "directive" {

go/ast/astutil/enclosing.go

@@ -113,7 +113,7 @@ func PathEnclosingInterval(root *ast.File, start, end token.Pos) (path []ast.Nod
 				// childrenOf elides the FuncType node beneath FuncDecl.
 				// Add it back here for TypeParams, Params, Results,
 				// all FieldLists). But we don't add it back for the "func" token
-				// even though it is is the tree at FuncDecl.Type.Func.
+				// even though it is the tree at FuncDecl.Type.Func.
 				if decl, ok := node.(*ast.FuncDecl); ok {
 					if fields, ok := child.(*ast.FieldList); ok && fields != decl.Recv {
 						path = append(path, decl.Type)

go/ssa/builder.go

@@ -138,7 +138,7 @@ type builder struct {
 	finished int // finished is the length of the prefix of fns containing built functions.
 
 	// The task of building shared functions within the builder.
-	// Shared functions are ones the the builder may either create or lookup.
+	// Shared functions are ones the builder may either create or lookup.
 	// These may be built by other builders in parallel.
 	// The task is done when the builder has finished iterating, and it
 	// waits for all shared functions to finish building.

go/types/typeutil/map_test.go

@@ -214,7 +214,7 @@ func Fb1[P any](x *P) {
 func Fb2[Q any](x *Q) {
 }
 
-// G1 and G2 are mutally recursive, and have identical methods.
+// G1 and G2 are mutually recursive, and have identical methods.
 type G1[P any] struct{
 	Field *G2[P]
 }

gopls/doc/index.md

@@ -67,7 +67,7 @@ ensure that Gopls is updated when a new stable version is released.
 
 After updating, you may need to restart running Gopls processes to
 observe the effect. Each client has its own way to restart the server.
-(On a UNIX machine, you can use the commmand `killall gopls`.)
+(On a UNIX machine, you can use the command `killall gopls`.)
 
 Learn more in the
 [advanced installation instructions](advanced.md#installing-unreleased-versions).

gopls/internal/analysis/unusedfunc/unusedfunc.go

@@ -52,7 +52,7 @@ import (
 //
 // Types (sans methods), constants, and vars are more straightforward.
 // For now we ignore enums (const decls using iota) since it is
-// commmon for at least some values to be unused when they are added
+// common for at least some values to be unused when they are added
 // for symmetry, future use, or to conform to some external pattern.
 
 //go:embed doc.go

gopls/internal/cache/analysis.go

@@ -1064,7 +1064,7 @@ func (act *action) exec(ctx context.Context) (any, *actionSummary, error) {
 		// Read file from snapshot, to ensure reads are consistent.
 		//
 		// TODO(adonovan): make the dependency analysis sound by
-		// incorporating these additional files into the the analysis
+		// incorporating these additional files into the analysis
 		// hash. This requires either (a) preemptively reading and
 		// hashing a potentially large number of mostly irrelevant
 		// files; or (b) some kind of dynamic dependency discovery

gopls/internal/cache/mod.go

@@ -509,7 +509,7 @@ func ResolvedVersion(module *packages.Module) string {
 	return module.Version
 }
 
-// ResolvedPath returns the the module path, which considers replace directive.
+// ResolvedPath returns the module path, which considers replace directive.
 func ResolvedPath(module *packages.Module) string {
 	if module.Replace != nil {
 		return module.Replace.Path

gopls/internal/golang/completion/completion.go

@@ -2589,7 +2589,7 @@ func inferExpectedResultTypes(c *completer, callNodeIdx int) []types.Type {
 	var expectedResults []types.Type
 
 	// Check the parents of the call node to extract the expected result types of the call signature.
-	// Currently reverse inferences are only supported with the the following parent expressions,
+	// Currently reverse inferences are only supported with the following parent expressions,
 	// however this list isn't exhaustive.
 	switch node := c.path[callNodeIdx+1].(type) {
 	case *ast.KeyValueExpr:
@@ -2671,7 +2671,7 @@ func expectedValueSpecType(pkg *cache.Package, node *ast.ValueSpec, pos token.Po
 }
 
 // expectedAssignStmtTypes analyzes the provided assignStmt, and checks
-// to see if the provided pos is within a RHS expresison. If so, it report
+// to see if the provided pos is within a RHS expression. If so, it report
 // the expected type of that expression, and the LHS type(s) to which it
 // is being assigned.
 func expectedAssignStmtTypes(pkg *cache.Package, node *ast.AssignStmt, pos token.Pos) (objType types.Type, assignees []types.Type) {

gopls/internal/golang/implementation.go

@@ -372,7 +372,7 @@ func implementsObj(info *types.Info, file *ast.File, pos token.Pos) (types.Objec
 // a function body. The global search index excludes such types
 // because reliably naming such types is hard.)
 //
-// Results are reported via the the yield function.
+// Results are reported via the yield function.
 func localImplementations(ctx context.Context, snapshot *cache.Snapshot, pkg *cache.Package, id *ast.Ident, rel methodsets.TypeRelation, yield implYieldFunc) error {
 	queryType, queryMethod := typeOrMethod(pkg.TypesInfo().Defs[id])
 	if queryType == nil {