]> Cypherpunks.ru repositories - gostls13.git/commitdiff
fmt: put back named results in ss.scanBasePrefix
authorRuss Cox <rsc@golang.org>
Thu, 7 Mar 2019 03:24:54 +0000 (22:24 -0500)
committerRuss Cox <rsc@golang.org>
Thu, 14 Mar 2019 00:08:15 +0000 (00:08 +0000)
CL 165619 removed these names when it removed
the use of the plain 'return'. But the names help for
documentation purposes even without being mentioned
directly in the function, so removing them makes the
code less readable. Put them back. I renamed found
to zeroFound to make the meaning clearer.

Change-Id: I1010931f08290af0b0ede7d21b1404c2eea196a0
Reviewed-on: https://go-review.googlesource.com/c/go/+/165899
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/fmt/scan.go

index fe6cbd477f717f9ebf43a3d0a50b757c8242fd7a..4554f17300b4a145ffe74bef4e32710204727dc5 100644 (file)
@@ -612,7 +612,7 @@ func (s *ss) scanRune(bitSize int) int64 {
 // scanBasePrefix reports whether the integer begins with a bas prefix
 // and returns the base, digit string, and whether a zero was found.
 // It is called only if the verb is %v.
-func (s *ss) scanBasePrefix() (int, string, bool) {
+func (s *ss) scanBasePrefix() (base int, digits string, zeroFound bool) {
        if !s.peek("0") {
                return 0, decimalDigits + "_", false
        }