Age | Commit message (Collapse) | Author |
|
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9315044
|
|
««« CL 9311043 / 53bc96b4c0c7
runtime: fix crash in select
runtime.park() can access freed select descriptor
due to a racing free in another thread.
See the comment for details.
Slightly modified version of dvyukov's CL 9259045.
No test yet. Before this CL, the test described in issue 5422
would fail about every 40 times for me. With this CL, I ran
the test 5900 times with no failures.
Fixes #5422.
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9311043
»»»
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9304044
|
|
««« CL 9305043 / 87762a7629b4
spec: fix incorrect example
Fixes #5430.
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9305043
»»»
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9308044
|
|
JSON feed
««« CL 9288045 / 5785ebd7acfb
doc: pull front page featured articles using new blog JSON feed
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/9288045
»»»
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9050046
|
|
««« CL 9120045 / e4f62df3e6c9
cmd/cgo: pass -Wsystem-headers when looking for errors
This works around a bug in GCC 4.8.0.
Fixes #5118.
R=golang-dev, r, minux.ma
CC=golang-dev
https://golang.org/cl/9120045
»»»
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9259047
|
|
««« CL 9159043 / f1ddc3ce3dfe
net: fix dial race on plan9 and windows
Fixes #5349.
R=golang-dev, lucio.dere, dsymonds, bradfitz, iant, adg, dave, r
CC=golang-dev
https://golang.org/cl/9159043
»»»
R=golang-dev
CC=golang-dev
https://golang.org/cl/9315043
|
|
guidelines for interface names
««« CL 9274043 / bbe324079abe
effective_go.html: be more accepting in the guidelines for interface names
Fixes #5421.
R=golang-dev, bradfitz, adg
CC=golang-dev
https://golang.org/cl/9274043
»»»
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9137045
|
|
|
|
R=golang-dev, adg, r
CC=golang-dev
https://golang.org/cl/9250043
|
|
The linker can generate split stack prolog when a textflag 7 function
makes an indirect function call. If it happens, badsignal() crashes
trying to dereference g.
Fixes #5337.
R=bradfitz, dave, adg, iant, r, minux.ma
CC=adonovan, golang-dev
https://golang.org/cl/9226043
|
|
Update #5236
Update #5402
This CL reduces gofmt's committed memory from 545864 KiB to 139568 KiB.
Note: Go 1.0.3 uses about 70MiB.
R=golang-dev, r, iant, nightlyone
CC=golang-dev
https://golang.org/cl/9245043
|
|
Found while debugging memory usage. Nobody accesses this field
anymore.
R=golang-dev, i.caught.air, adg, r
CC=golang-dev
https://golang.org/cl/9108043
|
|
R=golang-dev, dsymonds, r
CC=golang-dev
https://golang.org/cl/9244043
|
|
API change, but the old API is obviously wrong.
R=golang-dev, iant, r, rsc
CC=golang-dev
https://golang.org/cl/9157044
|
|
Allows Examples with KeyValue expressions to be playable in godoc.
During the traversal of the abstract syntax tree any KeyValueExpr Key.Name was incorrectly being added as an unresolved identifier.
Since this identifier could not be provided the Example was marked as unplayable.
This updates the AST traversal to skip Keys (but continue traversing the Values).
Example of problematic AST now fixed (see L99 where "UpperBound" was being added as a missing identifier):
81 . . . . . . . . . Values: []ast.Expr (len = 1) {
82 . . . . . . . . . . 0: *ast.UnaryExpr {
83 . . . . . . . . . . . OpPos: 12:19
84 . . . . . . . . . . . Op: &
85 . . . . . . . . . . . X: *ast.CompositeLit {
86 . . . . . . . . . . . . Type: *ast.SelectorExpr {
87 . . . . . . . . . . . . . X: *ast.Ident {
88 . . . . . . . . . . . . . . NamePos: 12:20
89 . . . . . . . . . . . . . . Name: "t_proto"
90 . . . . . . . . . . . . . }
91 . . . . . . . . . . . . . Sel: *ast.Ident {
92 . . . . . . . . . . . . . . NamePos: 12:41
93 . . . . . . . . . . . . . . Name: "BConfig"
94 . . . . . . . . . . . . . }
95 . . . . . . . . . . . . }
96 . . . . . . . . . . . . Lbrace: 12:79
97 . . . . . . . . . . . . Elts: []ast.Expr (len = 2) {
98 . . . . . . . . . . . . . 0: *ast.KeyValueExpr {
99 . . . . . . . . . . . . . . Key: *ast.Ident {
100 . . . . . . . . . . . . . . . NamePos: 13:3
101 . . . . . . . . . . . . . . . Name: "UpperBound"
102 . . . . . . . . . . . . . . }
103 . . . . . . . . . . . . . . Colon: 13:13
104 . . . . . . . . . . . . . . Value: *ast.CallExpr {
105 . . . . . . . . . . . . . . . Fun: *ast.SelectorExpr {
106 . . . . . . . . . . . . . . . . X: *ast.Ident {
107 . . . . . . . . . . . . . . . . . NamePos: 13:15
108 . . . . . . . . . . . . . . . . . Name: "proto"
109 . . . . . . . . . . . . . . . . }
110 . . . . . . . . . . . . . . . . Sel: *ast.Ident {
111 . . . . . . . . . . . . . . . . . NamePos: 13:21
112 . . . . . . . . . . . . . . . . . Name: "Float32"
113 . . . . . . . . . . . . . . . . }
R=adg
CC=gobot, golang-dev, gri
https://golang.org/cl/8569045
|
|
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/9229043
|
|
Fixes #5413
R=golang-dev, dave, adg
CC=golang-dev
https://golang.org/cl/9153045
|
|
R=golang-dev, i.caught.air, alexb, r
CC=golang-dev
https://golang.org/cl/9064044
|
|
R=golang-dev, minux.ma, r
CC=golang-dev
https://golang.org/cl/9086043
|
|
Fixes #5403.
R=golang-dev, minux.ma
CC=golang-dev
https://golang.org/cl/9100046
|
|
TBR=iant
CC=golang-dev
https://golang.org/cl/9048048
|
|
Fixes #5392.
R=iant, r
CC=golang-dev
https://golang.org/cl/9119043
|
|
This must have been from when "error" was a good variable
name for an "os.Error". But we use "err" these days.
R=golang-dev, dsymonds, r
CC=golang-dev
https://golang.org/cl/9132045
|
|
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9103046
|
|
As advised by iant. Fixes windows build.
R=golang-dev, r
CC=golang-dev, iant
https://golang.org/cl/9110044
|
|
"The usual conversions" bite again.
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/9103044
|
|
Include libc.h before bio.h in 8.c, because bio.h uses
the UTFmax enum, which is declared in libc.h, since
the recent switch to 21-bit runes in Plan 9.
The 5.c and 6.c files already includes libc.h.
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9040047
|
|
R=rsc, alexb, minux.ma, bradfitz, i.caught.air
CC=golang-dev
https://golang.org/cl/9082043
|
|
R=rsc, r
CC=golang-dev
https://golang.org/cl/9060045
|
|
This is needed for SWIG when linking in internal mode. In
internal mode if a symbol was cgo_import_static we used to
forget that it was also cgo_import_dynamic.
R=rsc, r
CC=golang-dev
https://golang.org/cl/9080043
|
|
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9047045
|
|
Fixes #5355.
R=golang-dev, mikioh.mikioh, bradfitz, r
CC=golang-dev
https://golang.org/cl/8966046
|
|
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/9025047
|
|
Manual undo due to later changes in doc/go1.1.html; cmd/go/test.bash still passes.
Rationale, from CL 8119049 review log:
This makes the 'go run' command different from every other command.
For example, 'go test' does not mean 'go test *.go'.
If we were going to handle the no arguments case in 'go run', I would hope that
it would scan the current directory to find a package just like 'go build' or
'go test' would, and then it would require that package to be 'package main',
and then it would run that package. This would make it match 'go test' and 'go
build' and 'go install' and so on. It would mean that if you are working on a
command in a directory that is 'go install'able, then 'go run' will run the
binary for you. The current CL does not accomplish that when build constraints
or file name constraints are involved.
For example, if I am working on a program like:
$ ls
main.go
main_386.s
main_arm.s
main_amd64.s
$
Then 'go run' will fail here because the .s files are ignored.
If instead I am working on a program like:
$ ls
main.go
main_386.go
main_arm.go
main_amd64.go
$
then 'go run' will fail because too many files are included.
I would like to see this command implemented so that it is compatible with the
other go subcommands. Since it is too late to do that for Go 1.1, I would like
to see this CL reverted, to preserve the option to do it better later.
R=golang-dev, iant, r
CC=golang-dev
https://golang.org/cl/8797049
|
|
I would like opinions on whether this is a good idea for 1.1.
On the one hand it's a moderately important issue. On the
other hand this introduces at least the possibility of
external linker errors due to the additional relocations and
it may be better to wait.
I'm fairly confident that the behaviour is unchanged when not
using an external linker.
Update #5221
This CL is tested lightly on 386 and amd64 and fixes the cases
I tested. I have not tested it on Darwin or Windows.
R=golang-dev, dave, daniel.morsing, rsc
CC=golang-dev
https://golang.org/cl/8858047
|
|
It's too hard to make portable just now.
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/9057043
|
|
Linux.
R=iant, iant, r, bradfitz
CC=golang-dev
https://golang.org/cl/8334044
|
|
exec_plan9.go too.
Those are in CL 8334044
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/9055043
|
|
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/9036046
|
|
PC-relative needs a signed offset; others need unsigned.
Also fix signedness of 32-bit relocation on Windows.
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/9039045
|
|
Update #5164
R=golang-dev, dave
CC=golang-dev
https://golang.org/cl/9049043
|
|
the issue.
R=golang-dev, dsymonds, dave
CC=golang-dev
https://golang.org/cl/9038043
|
|
Some 64-bit fields were run through 32-bit words, some counts were
not checked for overflow, and relocations must fit in 32 bits.
Tests to follow.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/9033043
|
|
A few places in the linker pushed 64-bit values through 32-bit holes,
including in relocation.
Clean them up, and check for a few other overflows as well.
Tests to follow.
R=dsymonds
CC=gobot, golang-dev
https://golang.org/cl/9032043
|
|
Also added docs for InvalidUTF8Error.
Fixes #5360.
R=golang-dev, adg, r
CC=golang-dev
https://golang.org/cl/8926046
|
|
Fixes #5311
R=golang-dev, bradfitz, iant
CC=golang-dev
https://golang.org/cl/8961050
|
|
runtime.setmg() calls another function (cgo_save_gm), so it must save
LR onto stack.
Re-enabled TestCthread test in misc/cgo/test.
Fixes #4863.
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9019043
|
|
Fixes #5350.
R=r, minux.ma
CC=golang-dev
https://golang.org/cl/8950043
|
|
R=golang-dev, minux.ma, r
CC=golang-dev
https://golang.org/cl/8545047
|
|
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9000043
|