| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
| |
R=gri
CC=golang-codereviews
https://codereview.appspot.com/104340043
|
|
|
|
|
|
|
| |
LGTM=iant
R=ruiu, iant
CC=golang-codereviews
https://codereview.appspot.com/107320044
|
|
|
|
|
|
| |
R=golang-codereviews
CC=golang-codereviews
https://codereview.appspot.com/103520044
|
|
|
|
|
|
|
| |
LGTM=ruiu
R=golang-codereviews, ruiu
CC=golang-codereviews
https://codereview.appspot.com/109100046
|
|
|
|
|
|
|
|
|
|
|
| |
This is a clone of 101370043, which I accidentally applied to the
release branch first.
No big deal, it needed to be applied there anyway.
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/108090043
|
|
|
|
|
|
|
| |
LGTM=minux
R=golang-codereviews, minux
CC=golang-codereviews
https://codereview.appspot.com/102340044
|
|
|
|
|
|
| |
R=rsc
CC=golang-codereviews
https://codereview.appspot.com/105930043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/103340046
|
|
|
|
|
|
|
|
|
|
| |
Contains a link to /lib/godoc/analysis/help.html
which is not yet live.
LGTM=r
R=r, adg
CC=golang-codereviews
https://codereview.appspot.com/88560044
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8168.
TBR=bradfitz
R=golang-codereviews
CC=golang-codereviews
https://codereview.appspot.com/107950043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8156.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/108840045
|
|
|
|
|
|
|
|
|
| |
align=middle is invalid; use align=center
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/106910044
|
|
|
|
|
|
|
|
|
|
|
| |
The current wording is reversed in 2 places.
Not sure how it got 4 LGTMs (mine was there as well).
Update issue 6242.
LGTM=dan.kortschak, r, rsc
R=golang-codereviews, 0xjnml, dan.kortschak, r, rsc
CC=golang-codereviews
https://codereview.appspot.com/101980047
|
|
|
|
|
|
|
|
|
|
|
| |
building on both 386 and amd64.
Fixes issue 8152.
LGTM=iant
R=iant
CC=golang-codereviews
https://codereview.appspot.com/102150046
|
|
|
|
|
|
|
| |
LGTM=r
R=khr, r
CC=golang-codereviews
https://codereview.appspot.com/103810044
|
|
|
|
|
|
|
| |
LGTM=adg
R=golang-codereviews, r, adg
CC=golang-codereviews
https://codereview.appspot.com/102020045
|
|
|
|
|
|
|
|
|
| |
Update issue 8112
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/95640043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For incomplete struct S, C.T and C.struct_S were interchangeable in Go 1.2
and earlier, because all incomplete types were interchangeable
(even C.struct_S1 and C.struct_S2).
CL 76450043, which fixed issue 7409, made different incomplete types
different from Go's point of view, so that they were no longer completely
interchangeable.
However, imprecision about C.T and C.struct_S - really the same
underlying C type - is the one behavior enabled by the bug that
is most likely to be depended on by existing cgo code.
Explicitly allow it, to keep that code working.
Fixes issue 7786.
LGTM=iant, r
R=golang-codereviews, iant, r
CC=golang-codereviews
https://codereview.appspot.com/98580046
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also made it extra clear for goto statements (even though label scopes
are already limited to the function defining a label).
Fixes issue 8040.
LGTM=r, rsc
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/99550043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The spec was unclear about whether blank methods should be
permitted in interface types. gccgo permits at most one, gc
crashes if there are more than one, go/types permits at most
one.
Discussion:
Since method sets of non-interface types never contain methods
with blank names (blank methods are never declared), it is impossible
to satisfy an interface with a blank method.
It is possible to declare variables of assignable interface types
(but not necessarily identical types) containing blank methods, and
assign those variables to each other, but the values of those
variables can only be nil.
There appear to be two "reasonable" alternatives:
1) Permit at most one blank method (since method names must be unique),
and consider it part of the interface. This is what appears to happen
now, with corner-case bugs. Such interfaces can never be implemented.
2) Permit arbitrary many blank methods but ignore them. This appears
to be closer to the handling of blank identifiers in declarations.
However, an interface type literal is not a declaration (it's a type
literal). Also, for struct types, blank identifiers are not ignored;
so the analogy with declarations is flawed.
Both these alternatives don't seem to add any benefit and are likely
(if only slightly) more complicated to explain and implement than
disallowing blank methods in interfaces altogether.
Fixes issue 6604.
LGTM=r, rsc, iant
R=r, rsc, ken, iant
CC=golang-codereviews
https://codereview.appspot.com/99410046
|
|
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/98460045
Committer: Brad Fitzpatrick <bradfitz@golang.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The spec did not specify the order in which
init() functions are called. Specify that
they are called in source order since we have
now also specified the initialization order
of independent variables.
While technically a language change, no
existing code could have relied on this,
so this should not break anything.
Per suggestion from rsc.
LGTM=r, iant
R=rsc, iant, r, ken
CC=golang-codereviews
https://codereview.appspot.com/98420046
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- split description of package initialization and
program execution
- better grouping of concerns in section on package
initialization
- more explicit definition of what constitues a
dependency
- removed language about constant dependencies -
they are computed at compile-time and not
initialized at run-time
- clarified that independent variables are initialized
in declaration order (rather than reference order)
Note that the last clarification is what distinguishes
gc and gccgo at the moment: gc uses reference order
(i.e., order in which variables are referenced in
initialization expressions), while gccgo uses declaration
order for independent variables.
Not a language change. But adopting this CL will
clarify what constitutes a dependency.
Fixes issue 6703.
LGTM=adonovan, r, iant, rsc
R=r, rsc, iant, ken, adonovan
CC=golang-codereviews
https://codereview.appspot.com/99020043
|
|
|
|
|
|
|
| |
LGTM=gri
R=gri
CC=golang-codereviews
https://codereview.appspot.com/99420045
|
|
|
|
|
|
|
| |
LGTM=minux.ma
R=adg, minux.ma
CC=golang-codereviews
https://codereview.appspot.com/95540045
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/93470043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7958.
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/91590044
|
|
|
|
|
|
|
|
|
| |
Update issue 8030
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/94680043
|
|
|
|
|
|
|
|
|
|
|
| |
Update issue 4714
LGTM=iant, minux.ma, rsc
R=rsc, iant, r, minux.ma
CC=golang-codereviews
https://codereview.appspot.com/100390043
Committer: Russ Cox <rsc@golang.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The move from 4kB to 8kB in Go 1.2 was to eliminate many stack split hot spots.
The move back to 4kB was predicated on copying stacks eliminating
the potential for hot spots.
Unfortunately, the fact that stacks do not copy 100% of the time means
that hot spots can still happen under the right conditions, and the slowdown
is worse now than it was in Go 1.2. There is a real program in issue 8030 that
sees about a 30x slowdown: it has a reflect call near the top of the stack
which inhibits any stack copying on that segment.
Go back to 8kB until stack copying can be used 100% of the time.
Fixes issue 8030.
LGTM=khr, dave, iant
R=iant, khr, r, bradfitz, dave
CC=golang-codereviews
https://codereview.appspot.com/92540043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8023.
LGTM=rsc
R=r, iant, ken, rsc
CC=golang-codereviews
https://codereview.appspot.com/98340043
|
|
|
|
|
|
|
|
|
|
|
| |
Discussion here: https://codereview.appspot.com/100490044/#msg14
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/98350043
Committer: Rob Pike <r@golang.org>
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/92440043
|
|
|
|
|
|
|
| |
LGTM=r
R=golang-codereviews, aram, 0intro, gobot, r
CC=golang-codereviews
https://codereview.appspot.com/100490044
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8007.
LGTM=r
R=gri, r
CC=golang-codereviews
https://codereview.appspot.com/91510044
|
|
|
|
|
|
|
| |
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/98290048
|
|
|
|
|
|
|
| |
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://codereview.appspot.com/93430044
|
|
|
|
|
|
|
|
|
| |
The vet check is in CL 10470044.
LGTM=bradfitz, r
R=r, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/91480044
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make it a little clearer how they are used, in particular that
it is not enough just to return a nil pointer on error, but also
to return an error value explaining the problem.
Fixes issue 1963.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/97360045
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- use previously defined terms (with links) throughout
- specify evaluation order more precisely (in particular,
the evaluation time of rhs expressions in receive cases
was not specified)
- added extra example case
Not a language change.
Description matches observed behavior of code compiled
with gc and gccgo.
Fixes issue 7669.
LGTM=iant, r, rsc
R=r, rsc, iant, ken, josharian
CC=golang-codereviews
https://codereview.appspot.com/91230043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change requires using SWIG version 3.0 or later. Earlier
versions of SWIG do not generate the pragmas required to use
the external linker.
Fixes issue 7155.
Fixes issue 7156.
LGTM=rsc
R=rsc
CC=golang-codereviews
https://codereview.appspot.com/97120046
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7901.
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/90930044
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- A channel may be used between any number of goroutines,
not just two.
- Replace "passing a value" (which is not further defined)
by "sending and receiving a value".
- Made syntax production more symmetric.
- Talk about unbuffered channels before buffered channels.
- Clarify what the comma,ok receive values mean (issue 7785).
Not a language change.
Fixes issue 7785.
LGTM=rsc, r, iant
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/94030045
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a clarification of what happens already.
Not a language change.
Fixes issue 7137.
LGTM=iant, r, rsc
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/96000044
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the underlying type of a type T is a boolean, numeric,
or string type, then T is also a boolean, numeric, or
string type, respectively.
Not a language change.
Fixes issue 7551.
LGTM=iant, rsc, robert.hencke, r
R=r, rsc, iant, ken, robert.hencke
CC=golang-codereviews
https://codereview.appspot.com/100130044
|
|
|
|
|
|
|
|
|
|
| |
Currently tip.golang.org leads to golang.org and
local godoc also leads to golang.org (when you don't have internet connectivity).
LGTM=crawshaw
R=golang-codereviews, crawshaw
CC=golang-codereviews
https://codereview.appspot.com/100200043
|
|
|
|
|
|
|
| |
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews, r
https://codereview.appspot.com/95010049
|
|
|
|
|
|
|
|
|
| |
We will serve downloads from here until we work out a better plan.
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/95980044
|
|
|
|
|
|
| |
R=rsc, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/100110043
|
|
|
|
|
|
|
|
|
| |
FreeBSD 10 is fully supported.
LGTM=r
R=golang-codereviews, bradfitz, r
CC=dave, golang-codereviews
https://codereview.appspot.com/99810044
|