Skip to content

G
golang
Switch branch/tag
  1. 03 Jul, 2018 4 commits
  3. 02 Jul, 2018 12 commits
  5. 01 Jul, 2018 2 commits
  7. 30 Jun, 2018 1 commit
  9. 29 Jun, 2018 16 commits
  11. 28 Jun, 2018 5 commits
    • Ian Lance Taylor's avatar
      reflect: remove struct tags from unexported types · 997d7a18
      Ian Lance Taylor authored 
      Before CL 4281055 in 2011, the reflect package was quite different.
rtype, then called commonType, was embedded in exported structs with
names like StructType. In order to avoid accidental conversions
between pointers to these public structs, which sometimes had
identical fields, the embedded commonType fields were tagged.

In CL 4281055 the formerly public structs were unexported, and all
access was done through the Type interface. At that point the field
tags in the reflect structs were no longer useful.

In Go 1.8 the language was changed to ignore struct field tags when
converting between types. This made the field tags in the reflect
structs doubly useless.

This CL simply removes them.

Fixes #20914

Change-Id: I9af4d6d0709276a91a6b6ee5323cad9dcd0cd0a0
Reviewed-on: https://go-review.googlesource.com/121475
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
      997d7a18
    • Robert Griesemer's avatar
      cmd/compile: don't crash in untyped expr to interface conversion · b410ce75
      Robert Griesemer authored 
      Fixes #24763.

Change-Id: Ibe534271d75b6961d00ebfd7d42c43a3ac650d79
Reviewed-on: https://go-review.googlesource.com/121335Reviewed-by: 's avatarMatthew Dempsky <mdempsky@google.com>
      b410ce75
    • Brad Fitzpatrick's avatar
      net/http: make Server.Shutdown treat new connections as idle after 5 seconds · a79fe535
      Brad Fitzpatrick authored 
      The Server distinguishes "new" vs "idle" connections. A TCP connection
from which no bytes have yet been written is "new". A connection that
has previously served a request and is in "keep-alive" state while
waiting for a second or further request is "idle".

The graceful Server.Shutdown historically only shut down "idle"
connections, with the assumption that a "new" connection was about to
read its request and would then shut down on its own afterwards.

But apparently some clients spin up connections and don't end up using
them, so we have something that's "new" to us, but browsers or other
clients are treating as "idle" to them.

This CL tweaks our heuristic to treat a StateNew connection as
StateIdle if it's been stuck in StateNew for over 5 seconds.

Fixes #22682

Change-Id: I01ba59a6ab67755ca5ab567041b1f54aa7b7da6f
Reviewed-on: https://go-review.googlesource.com/121419
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: 's avatarIan Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
      a79fe535
    • Russ Cox's avatar
      cmd/vet: fix ironic misuse of fmt.Sprintf · 0d52c144
      Russ Cox authored 
      Move badf helper into top-level function so that prints from buildtag.go
are once again themselves printf-format-checked by vet.
Also, fix implementation, which was missing a ... in the Sprintf call and
produced messages like:

/Users/rsc/x_test.go:1: +build comment must appear before package clause and be followed by a blank line%!(EXTRA []interface {}=[])

These were introduced in CL 111415.

Change-Id: I000af3a4e01dc99fc79c9146aa68a71dace1460f
Reviewed-on: https://go-review.googlesource.com/121300
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: 's avatarRob Pike <r@golang.org>
      0d52c144
    • Andrew Braunstein's avatar
      text/template/parse: fix a comment around the assign operator · 578b9617
      Andrew Braunstein authored 
      Fix a comment that misrepresented the Assign operator (=).

Rename: colon-equals -> equals.

Change-Id: I405b8acfb0bcd1b176a91a95f9bfb61a4e85815f
GitHub-Last-Rev: aec0bf594c63d7b015f88f97f9953ade976817a4
GitHub-Pull-Request: golang/go#26112
Reviewed-on: https://go-review.googlesource.com/121416Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
      578b9617