Merge branch 'master' into 12ya-patch

Author: 12ya

cmd/signature-fuzzer/internal/fuzz-generator/gen_test.go

@@ -112,7 +112,7 @@ func TestIsBuildable(t *testing.T) {
 	verb(1, "output is: %s\n", string(coutput))
 }
 
-// TestExhaustive does a series of code genreation runs, starting with
+// TestExhaustive does a series of code generation runs, starting with
 // (relatively) simple code and then getting progressively more
 // complex (more params, deeper structs, turning on additional
 // features such as address-taken vars and reflect testing). The

cmd/stringer/endtoend_test.go

@@ -286,7 +286,7 @@ func TestTestFiles(t *testing.T) {
 	}
 }
 
-// The -output flag cannot be used in combiation with matching types across multiple packages.
+// The -output flag cannot be used in combination with matching types across multiple packages.
 func TestCollidingOutput(t *testing.T) {
 	testenv.NeedsTool(t, "go")
 	stringer := stringerPath(t)

go/analysis/checker/example_test.go

@@ -69,7 +69,7 @@ func Example() {
 	// min=bufio max=unsafe
 }
 
-// minmaxpkg is a trival example analyzer that uses package facts to
+// minmaxpkg is a trivial example analyzer that uses package facts to
 // compute information from the entire dependency graph.
 var minmaxpkg = &analysis.Analyzer{
 	Name:      "minmaxpkg",

go/analysis/doc/suggested_fixes.md

@@ -79,7 +79,7 @@ These requirements guarantee that suggested fixes can be cleanly applied.
 Because a driver may only analyze, or be able to modify, the current package,
 we restrict edits to the current package. In general this restriction should
 not be a big problem for users because other packages might not belong to the
-same module and so will not be safe to modify in a singe change.
+same module and so will not be safe to modify in a single change.
 
 On the other hand, analyzers will not be required to produce gofmt-compliant
 code. Analysis drivers will be expected to apply gofmt to the results of

go/analysis/internal/checker/checker.go

@@ -99,7 +99,7 @@ func Run(args []string, analyzers []*analysis.Analyzer) (exitcode int) {
 	// without having to remember what code to return.
 	//
 	// TODO(adonovan): interpreting exit codes is like reading tea-leaves.
-	// Insted of wasting effort trying to encode a multidimensional result
+	// Instead of wasting effort trying to encode a multidimensional result
 	// into 7 bits we should just emit structured JSON output, and
 	// an exit code of 0 or 1 for success or failure.
 	exitAtLeast := func(code int) {

go/analysis/passes/asmdecl/asmdecl.go

@@ -57,7 +57,7 @@ type asmArch struct {
 	// include the first integer register and first floating-point register. Accessing
 	// any of them counts as writing to result.
 	retRegs []string
-	// writeResult is a list of instructions that will change result register implicity.
+	// writeResult is a list of instructions that will change result register implicitly.
 	writeResult []string
 	// calculated during initialization
 	sizes    types.Sizes

go/analysis/passes/loopclosure/loopclosure.go

@@ -88,7 +88,7 @@ func run(pass *analysis.Pass) (any, error) {
 		//
 		// TODO: consider allowing the "last" go/defer/Go statement to be followed by
 		// N "trivial" statements, possibly under a recursive definition of "trivial"
-		// so that that checker could, for example, conclude that a go statement is
+		// so that checker could, for example, conclude that a go statement is
 		// followed by an if statement made of only trivial statements and trivial expressions,
 		// and hence the go statement could still be checked.
 		forEachLastStmt(body.List, func(last ast.Stmt) {

go/analysis/passes/structtag/structtag.go

@@ -107,7 +107,7 @@ func checkCanonicalFieldTag(pass *analysis.Pass, field *types.Var, tag string, s
 
 	// Embedded struct. Nothing to do for now, but that
 	// may change, depending on what happens with issue 7363.
-	// TODO(adonovan): investigate, now that that issue is fixed.
+	// TODO(adonovan): investigate, now that issue is fixed.
 	if field.Anonymous() {
 		return
 	}

go/analysis/unitchecker/unitchecker_test.go

@@ -170,7 +170,7 @@ func _() {
 		// TODO(golang/go#65729): this is unsound: any extra
 		// logging by the child process (e.g. due to GODEBUG
 		// options) will add noise to stderr, causing the
-		// CombinedOutput to be unparseable as JSON. But we
+		// CombinedOutput to be unparsable as JSON. But we
 		// can't simply use Output here as some of the tests
 		// look for substrings of stderr. Rework the test to
 		// be specific about which output stream to match.

go/callgraph/vta/graph.go

@@ -658,7 +658,7 @@ func (b *builder) call(c ssa.CallInstruction) {
 }
 
 func addArgumentFlows(b *builder, c ssa.CallInstruction, f *ssa.Function) {
-	// When f has no paremeters (including receiver), there is no type
+	// When f has no parameters (including receiver), there is no type
 	// flow here. Also, f's body and parameters might be missing, such
 	// as when vta is used within the golang.org/x/tools/go/analysis
 	// framework (see github.com/golang/go/issues/50670).
@@ -803,7 +803,7 @@ func (b *builder) nodeFromVal(val ssa.Value) node {
 		return function{f: v}
 	case *ssa.Parameter, *ssa.FreeVar, ssa.Instruction:
 		// ssa.Param, ssa.FreeVar, and a specific set of "register" instructions,
-		// satisifying the ssa.Value interface, can serve as local variables.
+		// satisfying the ssa.Value interface, can serve as local variables.
 		return local{val: v}
 	default:
 		panic(fmt.Errorf("unsupported value %v in node creation", val))