- 12 Jul, 2013 3 commits
-
-
Russ Cox authored
Fixes #5853. R=ken2 CC=golang-dev https://golang.org/cl/11104044
-
Paul Borman authored
The json package cheerfully would marshal type S struct { IP net.IP } but would give an error when unmarshalling. This change allows any type whose concrete type is a byte slice to be unmarshalled from a string. Fixes #5086. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/11161044
-
Alex Brainman authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/11064044
-
- 11 Jul, 2013 12 commits
-
-
ChaiShushan authored
when the program is not main package, `go run x.go` can't return the link error message. so use `go run x.go` in instead `go build x.go`. Fixes #5865. R=golang-dev, adg CC=golang-dev https://golang.org/cl/11165043
-
Robert Griesemer authored
Fixes #5862. R=r CC=golang-dev https://golang.org/cl/11168043
-
Shenghou Ma authored
R=golang-dev CC=golang-dev https://golang.org/cl/11188043
-
Shenghou Ma authored
Fixes #3250. R=rsc CC=golang-dev https://golang.org/cl/10757044
-
Shenghou Ma authored
1. "int e;" is unused, generating "unused variable" error. 2. a->e was typed void *[2], but was accessed with *(int *)(a->e), this generated "dereferencing type-punned pointer will break strict-aliasing rules" error. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/11009043
-
Shenghou Ma authored
Update #5847 Summary: syscall: implement Sendfile for OpenBSD and NetBSD R=golang-dev, rsc, dave CC=golang-dev https://golang.org/cl/10980043
-
Shenghou Ma authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/10981043
-
Dmitriy Vyukov authored
runtime.newproc/ready are deliberately sloppy about waking new M's, they only ensure that there is at least 1 spinning M. Currently to compensate for that, schedule() checks if the current P has local work and there are no spinning M's, it wakes up another one. It does not work if goroutines do not call schedule. With this change a spinning M wakes up another M when it finds work to do. It's also not ideal, but it fixes the underutilization. A proper check would require to know the exact number of runnable G's, but it's too expensive to maintain. Fixes #5586. This is reincarnation of cl/9776044 with the bug fixed. The bug was due to code added after cl/9776044 was created: if(tick - (((uint64)tick*0x4325c53fu)>>36)*61 == 0 && runtime·sched.runqsize > 0) { runtime·lock(&runtime·sched); gp = globrunqget(m->p, 1); runtime·unlock(&runtime·sched); } If M gets gp from global runq here, it does not reset m->spinning. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/10743044
-
ChaiShushan authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/10855043
-
Dmitriy Vyukov authored
Currently it crashes as follows: fatal error: unknown pc ... goroutine 71698 [runnable]: runtime.racegoend() src/pkg/runtime/race.c:171 runtime.goexit() src/pkg/runtime/proc.c:1276 +0x9 created by runtime_test.testConcurrentReadsAfterGrowth src/pkg/runtime/map_test.go:264 +0x332 R=golang-dev, rsc CC=golang-dev https://golang.org/cl/10674047
-
Nigel Tao authored
R=r CC=andybons, golang-dev https://golang.org/cl/11148043
-
Russ Cox authored
If you compute the size by subtraction from the address of the next symbol, it helps to wait until the symbols have been sorted by address. R=golang-dev, r CC=golang-dev https://golang.org/cl/11143043
-
- 10 Jul, 2013 8 commits
-
-
Nigel Tao authored
R=r, andybons CC=andybons, golang-dev https://golang.org/cl/10977043
-
Robert Griesemer authored
R=khr CC=golang-dev https://golang.org/cl/11131043
-
Robert Griesemer authored
Instead, rely on the type checker. R=adonovan CC=golang-dev https://golang.org/cl/10826044
-
Andrew Gerrand authored
R=golang-dev, dsymonds, bradfitz CC=golang-dev https://golang.org/cl/11095043
-
Alex Brainman authored
Fixes #5783 R=golang-dev, r CC=golang-dev https://golang.org/cl/10956043
-
Robert Griesemer authored
The notion of a named type is crucial for the definition of type identity, assignability, definitions of methods. Explicitly introduce the notion with an extra sentence. Fixes #5682. R=r, rsc, iant CC=golang-dev https://golang.org/cl/11055043
-
Brad Fitzpatrick authored
R=golang-dev, adg CC=golang-dev https://golang.org/cl/11080043
-
Andrew Gerrand authored
Fixes #5843. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/11073043
-
- 09 Jul, 2013 5 commits
-
-
Rob Pike authored
Merging a couple of CLs into one, since they collided in my client and I'm lazy. 1) Fix up output in "go test -cover" case. We need to tell the testing package the name of the package being tested and the name of the package being covered. It can then sort out the report. 2) Filter out the _test.go files from coverage processing. We want to measure what the tests cover, not what's covered in the tests, The coverage for encoding/gob goes from 82.2% to 88.4%. There may be a cleaner way to do this - suggestions welcome - but ça suffit. Fixes #5810. R=rsc CC=golang-dev https://golang.org/cl/10868047
-
Robert Griesemer authored
Fixes #5787. R=r CC=golang-dev https://golang.org/cl/11057043
-
Nigel Tao authored
R=r, rsc, andybons CC=andybons, golang-dev https://golang.org/cl/10890045
-
Dave Cheney authored
Fix warning found by clang 3.3. R=rsc, r CC=golang-dev https://golang.org/cl/11022043
-
ChaiShushan authored
R=r CC=golang-dev https://golang.org/cl/10758044
-
- 08 Jul, 2013 1 commit
-
-
Andrew Gerrand authored
Fixes #5503. R=golang-dev, r CC=golang-dev https://golang.org/cl/10989043
-
- 07 Jul, 2013 1 commit
-
-
Shenghou Ma authored
CL 10869046 changed cmd/go to checkout master branch, so for "go get -u" to work, it must "git pull" instead of "git fetch". Added "--ff-only" so that it won't accidentally overwrite user changes. R=dsymonds CC=golang-dev https://golang.org/cl/10907043
-
- 05 Jul, 2013 5 commits
-
-
Daniel Morsing authored
Fixes #5809. R=golang-dev, dave, rsc, nigeltao CC=golang-dev https://golang.org/cl/10785043
-
Oliver Hookins authored
R=golang-dev, dave, adg CC=golang-dev https://golang.org/cl/10862045
-
Dave Cheney authored
Part 3 of several. * Linux has grown a SetsockoptByte. * SetsockoptIPMreqn is handled directly by syscall_linux.go and syscall_freebsd.go. R=golang-dev, mikioh.mikioh, r, bradfitz CC=golang-dev https://golang.org/cl/10775043
-
Andrew Gerrand authored
Generated by addca. R=gobot CC=golang-dev https://golang.org/cl/10868046
-
Nigel Tao authored
The lzw.NewReader doc comment says, "It is the caller's responsibility to call Close on the ReadCloser when finished reading." Thanks to Andrew Bonventre for noticing this. R=r, dsymonds, adg CC=andybons, golang-dev https://golang.org/cl/10821043
-
- 04 Jul, 2013 1 commit
-
-
Andrew Gerrand authored
R=dsymonds CC=golang-dev https://golang.org/cl/10933044
-
- 03 Jul, 2013 4 commits
-
-
Robert Griesemer authored
Instead, leave the error testing to the type checker, eventually. Fixes #5827. R=adonovan CC=golang-dev https://golang.org/cl/10917043
-
Brad Fitzpatrick authored
ReadMIMEHeader is used by net/http, net/mail, and mime/multipart. Don't do so many small allocations. Calculate up front how much we'll probably need. benchmark old ns/op new ns/op delta BenchmarkReadMIMEHeader 8433 7467 -11.45% benchmark old allocs new allocs delta BenchmarkReadMIMEHeader 23 14 -39.13% benchmark old bytes new bytes delta BenchmarkReadMIMEHeader 1705 1343 -21.23% R=golang-dev, r, iant, adg CC=golang-dev https://golang.org/cl/8179043
-
Brad Fitzpatrick authored
Generated by addca. R=gobot CC=golang-dev https://golang.org/cl/10840044
-
David Symonds authored
origin/master is always a remote branch, and it doesn't make sense to switch to a remote branch. master is the default branch that tracks it. R=adg CC=golang-dev, matt.jibson https://golang.org/cl/10869046
-