Age | Commit message (Collapse) | Author |
|
R=rsc
CC=golang-dev
https://golang.org/cl/4363043
|
|
Amazon S3 sends Transfer-Encoding "chunked"
on its 404 responses to HEAD requests for
missing objects.
We weren't ignoring the Transfer-Encoding
and were thus interpretting the subsequent
response headers as a chunk header from the
previous responses body (but a HEAD response
can't have a body)
R=rsc, adg
CC=golang-dev
https://golang.org/cl/4346050
|
|
A connection shouldn't be made available
for re-use until its body has been consumed.
(except in the case of pipelining, which isn't
implemented yet)
This CL fixes some issues seen with heavy load
against Amazon S3.
Subtle implementation detail: to prevent a race
with the client requesting a new connection
before previous one is returned, we actually
have to call putIdleConnection _before_ we
return from the final Read/Close call on the
http.Response.Body.
R=rsc, adg
CC=golang-dev
https://golang.org/cl/4351048
|
|
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/4315049
|
|
R=golang-dev, rsc1, peterGo, bsiegert, r, mattn
CC=golang-dev
https://golang.org/cl/4271057
|
|
The transport readLoop was waiting forever for the client to
read the non-existent body before proceeding to read the next
request.
R=rsc
CC=golang-dev
https://golang.org/cl/4357051
|
|
R=r, r2, rsc1, rsc, jacek.masiulaniec
CC=golang-dev
https://golang.org/cl/4344062
|
|
TBR=r
CC=golang-dev
https://golang.org/cl/4351047
|
|
The error will only occur for invalid patterns, but without this
error path there is no way to know that Glob has failed due to
an invalid pattern.
R=rsc
CC=golang-dev
https://golang.org/cl/4346044
|
|
Write never writes less than the buffer size and WriteString takes advantage
of the copy built-in to improve write efficiency.
R=rsc, ality, rog
CC=golang-dev
https://golang.org/cl/4344060
|
|
According to RFC 3986: "For consistency, URI producers
and normalizers should use uppercase hexadecimal digits
for all percent-encodings." Using lower case characters
makes it incompatible with Google APIs when signing OAuth requests.
R=golang-dev, rsc1, rsc
CC=golang-dev
https://golang.org/cl/4352044
|
|
R=golang-dev, r2
CC=golang-dev, mrjones
https://golang.org/cl/4348048
|
|
R=r
CC=golang-dev
https://golang.org/cl/4353047
|
|
Fixes #1637.
R=rsc, rh, msolo
CC=golang-dev
https://golang.org/cl/4317049
|
|
R=rsc
CC=golang-dev
https://golang.org/cl/4331050
|
|
Enables the use of https servers in tests.
R=agl, rsc, agl1
CC=golang-dev
https://golang.org/cl/4284063
|
|
Ideally we'd compile all example files during
the build, though.
Fixes #1660
R=r
CC=golang-dev
https://golang.org/cl/4358049
|
|
Fixes #1651.
R=rsc
CC=golang-dev
https://golang.org/cl/4300054
|
|
R=paulzhol, ality, r, fhs
CC=golang-dev
https://golang.org/cl/4316054
|
|
these were needed by packages in crypto/ and by io/ioutil
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/4350047
|
|
R=r, peterGo
CC=golang-dev
https://golang.org/cl/4344053
|
|
R=rsc, ality, r, r2
CC=golang-dev
https://golang.org/cl/4149046
|
|
CC=golang-dev
https://golang.org/cl/3816043
|
|
R=r, r2, rsc1
CC=golang-dev
https://golang.org/cl/4343045
|
|
Forgot this one in the previous CL
R=rsc, r
CC=golang-dev
https://golang.org/cl/4345043
|
|
R=ken2
CC=golang-dev
https://golang.org/cl/4316051
|
|
R=rsc
CC=golang-dev
https://golang.org/cl/4350043
|
|
Fixes #1610.
R=rsc
CC=golang-dev
https://golang.org/cl/4355042
|
|
R=golang-dev, adg, peterGo
CC=golang-dev
https://golang.org/cl/4274072
|
|
R=golang-dev, rsc1
CC=golang-dev
https://golang.org/cl/4347041
|
|
R=peterGo, rsc, Joe Poirier
CC=golang-dev
https://golang.org/cl/4280087
|
|
Since Go code can deadlock, this lets a testsuite driver set a
time limit for the test to run. This is simple but imperfect,
in that it only catches deadlocks in Go code, not in the
runtime scheduler.
R=r, rsc, iant2
CC=golang-dev
https://golang.org/cl/4326048
|
|
Also fix a bug: precision was in terms of bytes; should be runes.
Fixes #1652.
R=rsc, bradfitzgo, r2, bradfitzwork
CC=golang-dev
https://golang.org/cl/4280086
|
|
It was speculation.
Also replace one silly word with an English replacement.
R=golang-dev, bradfitzgo
CC=golang-dev
https://golang.org/cl/4339041
|
|
R=rsc
CC=golang-dev
https://golang.org/cl/4280079
|
|
Fix resolv_windows.go to support recent DNS-lookup changes
R=brainman, rsc1, rsc
CC=golang-dev
https://golang.org/cl/4300045
|
|
doc.go contains the details. The short story:
- command line is passed to the binary
- a new flag, -file, is needed to name files
- known flags have the "test." prefix added for convenience.
- gotest-specific flags are trimmed from the command line.
The effect should be that most existing uses are unaffected,
the ability to name files is still present, and it's nicer to use.
The downside is a lot more code in gotest.
Also allow a test to be called just Test.
R=rsc, niemeyer, rog, r2
CC=golang-dev
https://golang.org/cl/4307049
|
|
TBR=r
CC=golang-dev
https://golang.org/cl/4333048
|
|
R=r, r2
CC=golang-dev
https://golang.org/cl/4307047
|
|
This will make the fmt code easier to gofix
when the new reflect interface is ready.
R=r
CC=golang-dev
https://golang.org/cl/4324043
|
|
also: minor fix to parser
Note: gotest won't run the gotype test yet until
it permits TestXXX functions where XXX is empty.
R=r
CC=golang-dev
https://golang.org/cl/4300053
|
|
- don't consume '\n' as part of line comment
(otherwise grammars where '\n' are tokens won't
see them after a line comment)
- permit line comments to end in EOF
R=r
CC=golang-dev
https://golang.org/cl/4277089
|
|
This lets the test work for gccgo on other architectures.
R=r, rsc1
CC=golang-dev
https://golang.org/cl/4326044
|
|
R=r, rsc1, iant2
CC=golang-dev
https://golang.org/cl/4329044
|
|
R=gri
CC=golang-dev
https://golang.org/cl/4332045
|
|
R=gri
CC=golang-dev
https://golang.org/cl/4277087
|
|
As a special case, multi-line raw strings (i.e., strings in `` quotes)
were not indented if they were the only token on a line. This heuristic
was meant to improve formatting for multi-line raw strings where sub-
sequent lines are not indented at the level of the surrounding code.
Multiple people have complained about this. Removing the heuristic
again because it makes the formatting more regular, easier to under-
stand, and simplifies the implementation.
- manual changes to ebnf/ebnf_test.go for readability
- gofmt -w src misc
Fixes #1643.
R=r, rsc
CC=golang-dev
https://golang.org/cl/4307045
|
|
I'm in two minds as to whether this should be a function of gotest.
Tests that can flake out like this should be rare enough that we
needn't add more mechanism.
R=r
CC=golang-dev
https://golang.org/cl/4335042
|
|
R=rsc
CC=golang-dev
https://golang.org/cl/4271086
|
|
Failing at compile time requires that for each conversion
between two interface types the compiler compare the sets of
unexported methods to see if they come from different
packages. Since this test will fail approximately never on
real code, and since it can't catch all cases of the problem,
I don't think it's worth testing in the compiler. This CL
changes this test to look for a run-time panic rather than a
compile-time error.
R=gri, rsc1, iant2, rsc
CC=golang-dev
https://golang.org/cl/4332041
|