| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
People viewing this locally will not have a /s/ on their local godoc.
tip.golang.org doesn't have one either.
Also change all golang.org links to https, to avoid mixed content
warnings when viewing https://golang.org/.
Fixes issue 9028.
LGTM=bradfitz, r
R=r, bradfitz
CC=adg, golang-codereviews
https://codereview.appspot.com/168250043
|
|
|
|
|
|
|
| |
LGTM=r, adg
R=adg, r, 0xjnml, dr.volker.dobler
CC=golang-codereviews
https://codereview.appspot.com/166980044
|
|
|
|
|
|
|
| |
LGTM=r, rsc
R=r, rsc, adg
CC=golang-codereviews
https://codereview.appspot.com/168890043
|
|
|
|
|
|
|
|
|
| |
It doesn't simplify, because it wasn't even possible before.
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/164250043
|
|
|
|
|
|
|
| |
LGTM=adg
R=r, adg
CC=golang-codereviews
https://codereview.appspot.com/165890043
|
|
|
|
|
|
|
|
|
| |
First draft now complete.
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/170750043
|
|
|
|
|
|
|
| |
LGTM=iant, cmang
R=cmang, iant, rsc
CC=golang-codereviews
https://codereview.appspot.com/169760043
|
|
|
|
|
|
|
| |
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/165090043
|
|
|
|
|
|
|
| |
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/160660046
|
|
|
|
|
|
|
| |
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/166850043
|
|
|
|
|
|
|
|
|
|
|
| |
Also a few other minor changes.
Fixes issue 8712.
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/164150043
|
|
|
|
|
|
|
| |
LGTM=rsc, aram, minux
R=golang-codereviews, aram, minux, rsc
CC=golang-codereviews
https://codereview.appspot.com/162370045
|
|
|
|
|
|
|
| |
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/165800043
|
|
|
|
|
|
|
|
|
| |
still need to do internal and import comments
LGTM=adg
R=golang-codereviews, adg
CC=golang-codereviews
https://codereview.appspot.com/160600043
|
|
|
|
|
|
|
| |
LGTM=adg, rsc
R=golang-codereviews, adg, bradfitz, dave, rsc
CC=golang-codereviews
https://codereview.appspot.com/164090044
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a short introductory section saying what most Go
programmers really need to know, which is that you
shouldn't have to read this document to understand
the behavior of your program.
LGTM=bradfitz, adg, tracey.brendan, iant, rsc, dsymonds
R=golang-codereviews, bradfitz, tracey.brendan, adg, iant, rsc, dsymonds
CC=golang-codereviews
https://codereview.appspot.com/158500043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Not a language change.
This is simply documenting the status quo which permits
builtin function names to be parenthesized in calls; e.g.,
both
len(s)
and
(((len)))(s)
are accepted by all compilers and go/types.
Changed the grammar by merging the details of BuiltinCall
with ordinary Calls. Also renamed the Call production to
Arguments which more clearly identifies that part of the
grammar and also matches better with its counterpart on
the declaration side (Parameters).
The fact that the first argument can be a type (for builtins)
or cannot be a type (for regular function calls) is expressed
in the prose, no need to make the grammar more complicated.
Fixes issue 9001.
LGTM=iant, r, rsc
R=r, rsc, iant, ken, dave
CC=golang-codereviews
https://codereview.appspot.com/160570043
|
|
|
|
|
|
|
|
|
|
| |
Move the release notes into an HTML file.
Start writing the text.
LGTM=rsc
R=golang-codereviews, bradfitz, kamil.kisiel, tracey.brendan, rsc
CC=golang-codereviews
https://codereview.appspot.com/161350043
|
|
|
|
|
|
|
|
|
| |
Wrong article, one stylistic point that bothers someone (but not me).
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/156680043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/164770043
|
|
|
|
|
|
|
|
|
|
|
|
| |
The compatibility guideline needs to be clear about this even
though it means adding a clause that was not there from the
beginning. It has always been understood, so this isn't really
a change in policy, just in its expression.
LGTM=bradfitz, gri, rsc
R=golang-codereviews, bradfitz, gri, rsc
CC=golang-codereviews
https://codereview.appspot.com/162060043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/162140043
|
|
|
|
|
|
|
|
|
|
|
|
| |
Per suggestion from rsc as a result of the dicussion of
(abandoned) CL 153110044.
Fixes issue 7192.
LGTM=r, rsc, iant
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/163050043
|
|
|
|
|
|
|
| |
LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/163760043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/156430043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/161820043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8496.
LGTM=rsc, r, iant
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/148580043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/159800044
|
|
|
|
|
|
|
| |
LGTM=bradfitz, r
R=r, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/156730043
|
|
|
|
|
|
|
|
| |
Debugging the builders and need a commit to make them run.
R=adg
CC=golang-codereviews
https://codereview.appspot.com/149600043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/151490043
|
|
|
|
|
|
| |
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/149370043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/152160043
|
|
|
|
|
|
|
| |
Update issue 8750
CC=golang-codereviews
https://codereview.appspot.com/153950044
|
|
|
|
|
|
|
|
|
| |
This new text won't stop the whining but it might focus the whining a little more.
LGTM=adg
R=golang-codereviews, adg
CC=golang-codereviews
https://codereview.appspot.com/146680044
|
|
|
|
|
|
|
| |
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/149280044
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Not a language change.
Several inaccuracies were fixed:
1) A variable declaration may declare more than just one
variable.
2) Variable initialization follows the rules of assignments,
including n:1 assignments. The existing wording implied a 1:1
or n:n rule and generally was somewhat unspecific.
3) The rules for variable declarations with no types and
untyped initialization expressions had minor holes (issue 8088).
4) Clarified the special cases of assignments of untyped values
(we don't just have untyped constants, but also untyped bools,
e.g. from comparisons). The new wording is more direct.
To that end, introduced the notion of an untyped constant's
"default type" so that the same concept doesn't have to be
repeatedly introduced.
Fixes issue 8088.
LGTM=iant, r, rsc
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/142320043
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The existing spec rules on package initialization were
contradictory: They specified that 1) dependent variables
are initialized in dependency order, and 2) independent
variables are initialized in declaration order. This 2nd
rule cannot be satisfied in general. For instance, for
var (
c = b + 2
a = 0
b = 1
)
because of its dependency on b, c must be initialized after b,
leading to the partial order b, c. Because a is independent of
b but is declared before b, we end up with the order: a, b, c.
But a is also independent of c and is declared after c, so the
order b, c, a should also be valid in contradiction to a, b, c.
The new rules are given in form of an algorithm which outlines
initialization order explicitly.
gccgo and go/types already follow these rules.
Fixes issue 8485.
LGTM=iant, r, rsc
R=r, rsc, iant, ken, gordon.klaus, adonovan
CC=golang-codereviews
https://codereview.appspot.com/142880043
|
|
|
|
|
|
|
| |
LGTM=iant
R=golang-codereviews, bradfitz, iant
CC=golang-codereviews
https://codereview.appspot.com/150190043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/144660043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/143650043
|
|
|
|
|
|
| |
LGTM=r
R=r, rsc, iant, agl
https://codereview.appspot.com/142650045
|
|
|
|
|
|
|
| |
LGTM=0intro
R=golang-codereviews, 0intro
CC=golang-codereviews
https://codereview.appspot.com/150960043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7834.
LGTM=iant, rsc, r
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/148940044
|
|
|
|
|
|
|
|
|
| |
Fixes issue 7886.
LGTM=iant, r, rsc
R=r, iant, rsc, ken
CC=golang-codereviews
https://codereview.appspot.com/149010043
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/145890046
|
|
|
|
|
| |
CC=golang-codereviews
https://codereview.appspot.com/142450043
|
|
|
|
|
|
|
|
|
| |
The proposed text in the last CL had a comma that was missing from the submitted spec.
LGTM=gri
R=gri
CC=golang-codereviews
https://codereview.appspot.com/150720043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8107.
LGTM=iant, rsc, r
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/145960043
|
|
|
|
|
|
|
|
|
| |
Fixes issue 8202.
LGTM=r, bradfitz
R=r, josharian, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/148770043
|