Commits

Author Commit Message Labels Comments Date
Aram Hăvărneanu
runtime: add support for GOOS=solaris R=alex.brainman, dave, jsing, gobot, rsc CC=golang-codereviews https://codereview.appspot.com/35990043 Committer: Joel Sing <jsing@google.com>
Branches
default
Rob Pike
net/rpc: fix inconsistency in documentation of Service.Register Falsely claimed an old, no longer true condition that the first argument must be a pointer. Fixes issue 6697 R=golang-codereviews, bradfitz CC=golang-codereviews https://codereview.appspot.com/53480043
Branches
default
Dave Cheney
net: skip TestDualStackTCPListener in short mode Update issue 5001 This test is flakey on linux servers and fails otherwise good builds. Mikio has some proposals to fix the test, but they require additional plumbing. In the meantime, disable this test in -short mode so it will run during the full net test suite, but not during builder ci. R=golang-codereviews, iant CC=golang-codereviews https://codereview.appspot.com/53410043
Branches
default
Rob Pike
time: break parse and formatting tests into a separate source file No changes, just rearrangement. The tests were in need of a little housekeeping. R=golang-codereviews, iant CC=golang-codereviews https://codereview.appspot.com/53400043
Branches
default
Brad Fitzpatrick
syscall: add Flock_t on Linux Matches Darwin and the BSDs. This means leveldb-go, kv, Camlistore, etc can stop defining these structs on Linux by hand. Update Issue 7059 R=golang-codereviews, dave, iant CC=golang-codereviews https://codereview.appspot.com/53350043
Branches
default
Keith Randall
reflect: Remove imprecise techniques from channel/select operations. Reflect used to communicate to the runtime using interface words, which is bad for precise GC because sometimes iwords hold a pointer and sometimes they don't. This change rewrites channel and select operations to always pass pointers to the runtime. reflect.Select gets somewhat more expensive, as we now do an allocation per receive case instead of one allocation whose size is the max of all the…
Branches
default
Brad Fitzpatrick
net/http: don't allow Content-Type or body on 204 and 1xx Status codes 204, 304, and 1xx don't allow bodies. We already had a function for this, but we were hard-coding just 304 (StatusNotModified) in a few places. Use the function instead, and flesh out tests for all codes. Fixes Issue 6685 R=golang-codereviews, r CC=golang-codereviews https://codereview.appspot.com/53290044
Branches
default
Kamil Kisiel
net/smtp: add examples R=golang-codereviews, bradfitz CC=golang-codereviews https://codereview.appspot.com/8274046 Committer: Brad Fitzpatrick <bradfitz@golang.org>
Branches
default
Brad Fitzpatrick
net/http: cache transport environment lookup Apparently this is expensive on Windows. Fixes Issue 7020 R=golang-codereviews, alex.brainman, mattn.jp, dvyukov CC=golang-codereviews https://codereview.appspot.com/52840043
Branches
default
Rob Pike
fmt: fix bug printing large zero-padded hexadecimal We forgot to include the width of "0x" when computing the crossover from internal buffer to allocated buffer. Also add a helper function to the test for formatting large zero-padded test strings. Fixes issue 6777. R=golang-codereviews, iant CC=golang-codereviews https://codereview.appspot.com/50820043
Branches
default
Russ Cox
cmd/gc: handle non-escaping address-taken variables better This CL makes the bitmaps a little more precise about variables that have their address taken but for which the address does not escape to the heap, so that the variables are kept in the stack frame rather than allocated on the heap. The code before this CL handled these variables by treating every return statement as using every such variable and depending on liveness analysis to essentially tre…
Branches
default
Russ Cox
cmd/gc: fix race build Missed this case in CL 51010045. TBR=khr CC=golang-codereviews https://codereview.appspot.com/53200043
Branches
default
Aram Hăvărneanu
runtime: address jsing's review points
Aram Hăvărneanu
merge upstream
Dmitriy Vyukov
runtime: output how long goroutines are blocked Example of output: goroutine 4 [sleep for 3 min]: time.Sleep(0x34630b8a000) src/pkg/runtime/time.goc:31 +0x31 main.func·002() block.go:16 +0x2c created by main.main block.go:17 +0x33 Full program and output are here: http://play.golang.org/p/NEZdADI3Td Fixes issue 6809. R=golang-codereviews, khr, kamil.kisiel, bradfitz, rsc CC=golang-codereviews https://codereview.appspot.com/50420043
Branches
default
Dmitriy Vyukov
runtime: use lock-free ring for work queues Use lock-free fixed-size ring for work queues instead of an unbounded mutex-protected array. The ring has single producer and multiple consumers. If the ring overflows, work is put onto global queue. benchmark old ns/op new ns/op delta BenchmarkMatmult 7 5 -18.12% BenchmarkMatmult-4 2 2 -18.98% BenchmarkMatmult-16 1 0 -12.84% BenchmarkCreateGoroutines 105 88 -16.10% BenchmarkCreateGoroutines-4 376 219 -41.76% BenchmarkCreateGoroutines-16 …
Branches
default
Keith Randall
reflect: add precise GC info for Call argument frame. Give proper types to the argument/return areas allocated for reflect calls. Avoid use of iword to manipulate receivers, which may or may not be pointers. Update issue 6490 R=rsc CC=golang-codereviews https://codereview.appspot.com/52110044
Branches
default
Brad Fitzpatrick
net/http: add disabled test for Body Read/Close lock granularity Update Issue 7121 R=golang-codereviews, gobot, dsymonds CC=golang-codereviews https://codereview.appspot.com/51750044
Branches
default
Robert Griesemer
go/scanner: report too short escape sequences Generally improve error messages for escape sequences. R=adonovan CC=golang-codereviews https://codereview.appspot.com/49430046
Branches
default
Rob Pike
cmd/pack: rewrite in Go Replace the pack command, a C program, with a clean reimplementation in Go. It does not need to reproduce the full feature set and it is no longer used by the build chain, but has a role in looking inside archives created by the build chain directly. Since it's not in C, it is no longer build by dist, so remove it from cmd/dist and make it a "tool" in cmd/go terminology. Fixes issue 2705 R=rsc, dave, minux.ma, josharian CC=golang-codereviews https://codereview.appspot.com/52310044
Branches
default
Dmitriy Vyukov
runtime: fix data race in GC Fixes issue 5139. Update issue 7065. R=golang-codereviews, bradfitz, minux.ma CC=golang-codereviews https://codereview.appspot.com/52090045
Branches
default
Brad Fitzpatrick
net/http: return UnexpectedEOF instead of EOF on truncated resposne Fixes Issue 6564 R=golang-codereviews, r CC=golang-codereviews https://codereview.appspot.com/52420043
Branches
default
minux
runtime/debug: force GC after setting of GCPercent to make it effective. See also discussion in CL 51010045. R=golang-codereviews, r CC=golang-codereviews https://codereview.appspot.com/52230043
Branches
default
Rob Pike
cmd/ld: document the -w flag, which disables DWARF generation R=golang-codereviews, iant CC=golang-codereviews https://codereview.appspot.com/52360044
Branches
default
Rob Pike
spec: tighten the wording around . imports Make it clear that if you do a . import, you cannot use a qualified identifier. R=gri CC=golang-codereviews https://codereview.appspot.com/52390043
Branches
default
Rob Pike
reflect: better document the tri-state for TryRecv R=rsc, iant CC=golang-codereviews https://codereview.appspot.com/52360043
Branches
default
Keith Randall
runtime: Change size of map iter offset so 32-bit version compiles cleanly. R=golang-codereviews, minux.ma CC=golang-codereviews https://codereview.appspot.com/52310043
Branches
default
Michael Kelly
net/http: escape contents of the directory indexes generated by FileServer Previously, filenames containing special characters could: 1) Escape the <a> tag, with a file called something like: ">foo 2) Break the links in the index by prematurely ending the path portion of the url, with a file called: foo?bar In order to avoid a forbidden dependency on the html package, I'm using htmlReplacer from net/http/server.go, which is equivalent…
Branches
default
Josh Bleecher Snyder
runtime: change map iteration randomization to use intra-bucket offset Map iteration previously started from a random bucket, but walked each bucket from the beginning. Now, iteration always starts from the first bucket and walks each bucket starting at a random offset. For performance, the random offset is selected at the start of iteration and reused for each bucket. Iteration over a map with 8 or fewer elements--a single bucket--will now be non-deterministic. The…
Branches
default
Brad Fitzpatrick
undo CL 47560044 / 40a37153a550 Still work to do. See http://golang.org/issue/7125 ««« original CL description net/http/cookiejar: document format of domain in PublicSuffix Document what values a PublicSuffixList must accept as a domain in a call to PublicSuffix. R=bradfitz, nigeltao CC=golang-codereviews https://codereview.appspot.com/47560044 Committer: Brad Fitzpatrick <bradfitz@golang.org> »»» R=golang-codereviews, minu…
Branches
default
  1. Prev
  2. Next