Rerun gofumpt with a modern version, to adapt the 1.19 GoDoc changes

Author: flimzy

build/build.go

@@ -72,9 +72,9 @@ func NewBuildContext(installSuffix string, buildTags []string) XContext {
 // In the directory containing the package, .go and .inc.js files are
 // considered part of the package except for:
 //
-//    - .go files in package documentation
-//    - files starting with _ or . (likely editor temporary files)
-//    - files with build constraints not satisfied by the context
+//   - .go files in package documentation
+//   - files starting with _ or . (likely editor temporary files)
+//   - files with build constraints not satisfied by the context
 //
 // If an error occurs, Import returns a non-nil error and a nil
 // *PackageData.

compiler/linkname.go

@@ -26,10 +26,10 @@ type GoLinkname struct {
 // This is a logical equivalent of a symbol name used by traditional linkers.
 // The following properties should hold true:
 //
-//  - Each named symbol within a program has a unique SymName.
-//  - Similarly named methods of different types will have different symbol names.
-//  - The string representation is opaque and should not be attempted to reversed
-//    to a struct form.
+//   - Each named symbol within a program has a unique SymName.
+//   - Similarly named methods of different types will have different symbol names.
+//   - The string representation is opaque and should not be attempted to reversed
+//     to a struct form.
 type SymName struct {
 	PkgPath string // Full package import path.
 	Name    string // Symbol name.
@@ -85,12 +85,12 @@ func (n SymName) IsMethod() (recv string, method string, ok bool) {
 //
 // GopherJS directive support has the following limitations:
 //
-//  - External linkname must be specified.
-//  - The directive must be applied to a package-level function or method (variables
-//    are not supported).
-//  - The local function referenced by the directive must have no body (in other
-//    words, it can only "import" an external function implementation into the
-//    local scope).
+//   - External linkname must be specified.
+//   - The directive must be applied to a package-level function or method (variables
+//     are not supported).
+//   - The local function referenced by the directive must have no body (in other
+//     words, it can only "import" an external function implementation into the
+//     local scope).
 func parseGoLinknames(fset *token.FileSet, pkgPath string, file *ast.File) ([]GoLinkname, error) {
 	var errs ErrorList = nil
 	var directives []GoLinkname

compiler/natives/src/math/big/big.go

@@ -4,5 +4,6 @@
 package big
 
 // TODO: This is a workaround for https://github.com/gopherjs/gopherjs/issues/652.
-//       Remove after that issue is resolved.
+//
+//	Remove after that issue is resolved.
 type Word uintptr

compiler/utils.go

@@ -84,13 +84,18 @@ func (fc *funcContext) Delayed(f func()) {
 // tuple elements.
 //
 // For example, for functions defined as:
-//   func a() (int, string) {return 42, "foo"}
-//   func b(a1 int, a2 string) {}
+//
+//	func a() (int, string) {return 42, "foo"}
+//	func b(a1 int, a2 string) {}
+//
 // ...the following statement:
-//     b(a())
+//
+//	b(a())
+//
 // ...will be transformed into:
-//     _tuple := a()
-//     b(_tuple[0], _tuple[1])
+//
+//	_tuple := a()
+//	b(_tuple[0], _tuple[1])
 func (fc *funcContext) expandTupleArgs(argExprs []ast.Expr) []ast.Expr {
 	if len(argExprs) != 1 {
 		return argExprs
@@ -513,24 +518,24 @@ func isBlank(expr ast.Expr) bool {
 //
 // For example, consider a Go type:
 //
-// 		 type SecretInt int
-//     func (_ SecretInt) String() string { return "<secret>" }
+//			 type SecretInt int
+//	    func (_ SecretInt) String() string { return "<secret>" }
 //
-//     func main() {
-//       var i SecretInt = 1
-//       println(i.String())
-//     }
+//	    func main() {
+//	      var i SecretInt = 1
+//	      println(i.String())
+//	    }
 //
 // For this example the compiler will generate code similar to the snippet below:
 //
-//     SecretInt = $pkg.SecretInt = $newType(4, $kindInt, "main.SecretInt", true, "main", true, null);
-//     SecretInt.prototype.String = function() {
-//       return "<secret>";
-//     };
-//     main = function() {
-//       var i = 1;
-//       console.log(new SecretInt(i).String());
-//     };
+//	SecretInt = $pkg.SecretInt = $newType(4, $kindInt, "main.SecretInt", true, "main", true, null);
+//	SecretInt.prototype.String = function() {
+//	  return "<secret>";
+//	};
+//	main = function() {
+//	  var i = 1;
+//	  console.log(new SecretInt(i).String());
+//	};
 //
 // Note that the generated code assigns a primitive "number" value into i, and
 // only boxes it into an object when it's necessary to access its methods.
@@ -699,13 +704,12 @@ func encodeIdent(name string) string {
 //
 // For example:
 //
-// 	"my_name" -> ".my_name"
-// 	"my name" -> `["my name"]`
+//	"my_name" -> ".my_name"
+//	"my name" -> `["my name"]`
 //
 // For more information about JavaScript property accessors and identifiers, see
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Property_Accessors and
 // https://developer.mozilla.org/en-US/docs/Glossary/Identifier.
-//
 func formatJSStructTagVal(jsTag string) string {
 	for i, r := range jsTag {
 		ok := unicode.IsLetter(r) || (i != 0 && unicode.IsNumber(r)) || r == '$' || r == '_'

js/js.go

@@ -2,25 +2,25 @@
 //
 // Use MakeWrapper to expose methods to JavaScript. Use MakeFullWrapper to expose methods AND fields to JavaScript. When passing values directly, the following type conversions are performed:
 //
-//  | Go type               | JavaScript type       | Conversions back to interface{} |
-//  | --------------------- | --------------------- | ------------------------------- |
-//  | bool                  | Boolean               | bool                            |
-//  | integers and floats   | Number                | float64                         |
-//  | string                | String                | string                          |
-//  | []int8                | Int8Array             | []int8                          |
-//  | []int16               | Int16Array            | []int16                         |
-//  | []int32, []int        | Int32Array            | []int                           |
-//  | []uint8               | Uint8Array            | []uint8                         |
-//  | []uint16              | Uint16Array           | []uint16                        |
-//  | []uint32, []uint      | Uint32Array           | []uint                          |
-//  | []float32             | Float32Array          | []float32                       |
-//  | []float64             | Float64Array          | []float64                       |
-//  | all other slices      | Array                 | []interface{}                   |
-//  | arrays                | see slice type        | see slice type                  |
-//  | functions             | Function              | func(...interface{}) *js.Object |
-//  | time.Time             | Date                  | time.Time                       |
-//  | -                     | instanceof Node       | *js.Object                      |
-//  | maps, structs         | instanceof Object     | map[string]interface{}          |
+//	| Go type               | JavaScript type       | Conversions back to interface{} |
+//	| --------------------- | --------------------- | ------------------------------- |
+//	| bool                  | Boolean               | bool                            |
+//	| integers and floats   | Number                | float64                         |
+//	| string                | String                | string                          |
+//	| []int8                | Int8Array             | []int8                          |
+//	| []int16               | Int16Array            | []int16                         |
+//	| []int32, []int        | Int32Array            | []int                           |
+//	| []uint8               | Uint8Array            | []uint8                         |
+//	| []uint16              | Uint16Array           | []uint16                        |
+//	| []uint32, []uint      | Uint32Array           | []uint                          |
+//	| []float32             | Float32Array          | []float32                       |
+//	| []float64             | Float64Array          | []float64                       |
+//	| all other slices      | Array                 | []interface{}                   |
+//	| arrays                | see slice type        | see slice type                  |
+//	| functions             | Function              | func(...interface{}) *js.Object |
+//	| time.Time             | Date                  | time.Time                       |
+//	| -                     | instanceof Node       | *js.Object                      |
+//	| maps, structs         | instanceof Object     | map[string]interface{}          |
 //
 // Additionally, for a struct containing a *js.Object field, only the content of the field will be passed to JavaScript and vice versa.
 package js

nosync/once.go

@@ -8,21 +8,23 @@ type Once struct {
 
 // Do calls the function f if and only if Do is being called for the
 // first time for this instance of Once. In other words, given
-// 	var once Once
+//
+//	var once Once
+//
 // if once.Do(f) is called multiple times, only the first call will invoke f,
 // even if f has a different value in each invocation.  A new instance of
 // Once is required for each function to execute.
 //
 // Do is intended for initialization that must be run exactly once.  Since f
 // is niladic, it may be necessary to use a function literal to capture the
 // arguments to a function to be invoked by Do:
-// 	config.once.Do(func() { config.init(filename) })
+//
+//	config.once.Do(func() { config.init(filename) })
 //
 // If f causes Do to be called, it will panic.
 //
 // If f panics, Do considers it to have returned; future calls of Do return
 // without calling f.
-//
 func (o *Once) Do(f func()) {
 	if o.done {
 		return

nosync/pool.go

@@ -28,7 +28,6 @@ package nosync
 // not a suitable use for a Pool, since the overhead does not amortize well in
 // that scenario. It is more efficient to have such objects implement their own
 // free list.
-//
 type Pool struct {
 	store []interface{}
 	New   func() interface{}

tests/gorepo/run.go

@@ -11,7 +11,7 @@
 //
 // To run manually with summary, verbose output, and full stack traces of of known failures:
 //
-// 	go run run.go -summary -v -show_known_fails
+//	go run run.go -summary -v -show_known_fails
 //
 // TODO(bradfitz): docs of some sort, once we figure out how we're changing
 // headers of files
@@ -45,7 +45,6 @@ import (
 // GOPHERJS: Known test fails for GopherJS compiler.
 //
 // TODO: Reduce these to zero or as close as possible.
-//
 var knownFails = map[string]failReason{
 	"fixedbugs/bug114.go":     {desc: "fixedbugs/bug114.go:15:27: B32 (untyped int constant 4294967295) overflows int"},
 	"fixedbugs/bug242.go":     {desc: "bad map check 13 false false Error: fail"},

tests/runtime_test.go

@@ -23,14 +23,14 @@ func Test_parseCallFrame(t *testing.T) {
 			want:  "foo https://gopherjs.github.io/playground/playground.js 102 11836",
 		},
 		{
-			name: "Chrome 96, anonymous eval",
+			name:  "Chrome 96, anonymous eval",
 			input: "	at eval (<anonymous>)",
-			want: "eval <anonymous> 0 0",
+			want:  "eval <anonymous> 0 0",
 		},
 		{
-			name: "Chrome 96, anonymous Array.forEach",
+			name:  "Chrome 96, anonymous Array.forEach",
 			input: "	at Array.forEach (<anonymous>)",
-			want: "Array.forEach <anonymous> 0 0",
+			want:  "Array.forEach <anonymous> 0 0",
 		},
 		{
 			name:  "Chrome 96, file location only",