Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Use a count of -1 for infinity. Ditto for Replace.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1704044
Committer: Rob Pike <r@golang.org>
|
|
R=golang-dev, rsc
CC=golang-dev
http://codereview.appspot.com/1665047
Committer: Russ Cox <rsc@golang.org>
|
|
methods when looking up names.
Fixes issue 764.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1170041
|
|
Rather than confuse people by linking to a spec we
don't fully support, let them just use the package docs.
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/1016041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/982043
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/953041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/824049
|
|
note that sortmain.go has been run through hg gofmt;
only the formatting of the day initializers changed.
i'm happy to revert that formatting if you'd prefer.
stop on error in doc/progs/run
R=r
CC=golang-dev
http://codereview.appspot.com/850041
|
|
into a string of decimal numbers.
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/624041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/391041
|
|
- gofmt -w src misc
- looking for feedback
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/223076
|
|
Adds tests and fixes for two cases that fail with the current release.
R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/217115
Committer: Rob Pike <r@golang.org>
|
|
gofmt -w -r 'strings.Bytes(a) -> []byte(a)' src/cmd src/pkg test/bench
gofmt -w -r 'strings.Runes(a) -> []int(a)' src/cmd src/pkg test/bench
delete unused imports
R=r
CC=golang-dev
http://codereview.appspot.com/224062
|
|
- add extra test cases to go/printer tests
- apply gofmt to src and misc
R=rsc
CC=golang-dev
http://codereview.appspot.com/223041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/197043
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/186169
|
|
Fixes issue 501.
R=rsc
CC=golang-dev
http://codereview.appspot.com/181183
|
|
Fixes issue 309.
R=rsc
CC=golang-dev
http://codereview.appspot.com/181044
|
|
Fixes issue 389.
R=rsc
CC=golang-dev
http://codereview.appspot.com/180061
|
|
parsing and printing to new syntax.
Use -oldparser to parse the old syntax,
use -oldprinter to print the old syntax.
2) Change default gofmt formatting settings
to use tabs for indentation only and to use
spaces for alignment. This will make the code
alignment insensitive to an editor's tabwidth.
Use -spaces=false to use tabs for alignment.
3) Manually changed src/exp/parser/parser_test.go
so that it doesn't try to parse the parser's
source files using the old syntax (they have
new syntax now).
4) gofmt -w src misc test/bench
5th and last set of files.
R=rsc
CC=golang-dev
http://codereview.appspot.com/180050
|
|
R=rsc
http://codereview.appspot.com/172041
|
|
* diagnose template not created with New
(current code just crashes)
* write []byte uninterpreted
(current code writes fmt format: "[65 65 65 65]")
R=r
CC=golang-dev
http://codereview.appspot.com/161075
|
|
- removed New(len int) in favor of new(Vector).Resize(len, cap)
- removed Init(len int) in favor of Resize(len, cap)
- runs all.bash
Fixes issue 294.
R=rsc, r, r1
http://codereview.appspot.com/157143
|
|
the bash scripts and makefiles for building go didn't take into account
the fact $GOROOT / $GOBIN could both be directories containing whitespaces,
and was not possible to build it in such a situation.
this commit adjusts the various makefiles/scripts to make it aware of that
possibility, and now it builds successfully when using a path with whitespaces
as well.
Fixes issue 115.
R=rsc, dsymonds1
http://codereview.appspot.com/157067
Committer: Russ Cox <rsc@golang.org>
|
|
R=r, gri
CC=golang-dev
http://codereview.appspot.com/156115
|
|
Allows the developer to pass a map either by itself for
evaluation, or inside a struct. Access to data inside
maps is identical to the current system for structs, ie.
-Psuedocode-
mp map[string]string = {
"header" : "A fantastic header!",
"footer" : "A not-so-fantastic footer!",
}
template.Execute(mp)
...can be accessed using {header} and {footer} in
the template. Similarly, for maps inside structs:
type s struct {
mp map[string]string,
}
s1 = new s
s1.mp["header"] = "A fantastic header!";
template.Execute(s1)
...is accessed using {mp.header}. Multi-maps, ie.
map[string](map[string]string) and maps of structs
containing more maps are unsupported, but then, I'm
not even sure if that's supported by the language.
Map elements can be of any type that can be written
by the formatters. Keys should really only be strings.
Fixes issue 259.
R=r, rsc
http://codereview.appspot.com/157088
Committer: Rob Pike <r@golang.org>
|
|
rsc's algorithm
- applied gofmt -w misc src
- partial CL (last chunk)
R=rsc, r
http://go/go-review/1024041
|
|
R=rsc, r
http://go/go-review/1025029
|
|
HTML vs Html, URL vs Url, HTTP vs Http,
current source is 6:1 in favor of the former,
so change instances of the latter.
R=r
CC=go-dev
http://go/go-review/1024026
|
|
- enabled for function declarations (not just function literals)
- applied gofmt -w $GOROOT/src
(look for instance at src/pkg/debug/elf/elf.go)
R=r, rsc
CC=go-dev
http://go/go-review/1026006
|
|
in special cases
- re-gofmt'ing of some files that are now improved
R=r, rsc
http://go/go-review/1023003
|
|
R=r
http://go/go-review/1018064
|
|
(replacement for CLs 1017039, 1017041, 1017040, 1018054)
R=r
http://go/go-review/1018060
|
|
R=rsc
http://go/go-review/1016041
|
|
R=rsc
http://go/go-review/1017025
|
|
R=gri
http://go/go-review/1017014
|
|
also fix a error-printing bug in godoc.
R=gri
http://go/go-review/1016030
|
|
fix up one usage to take advantage.
R=rsc
CC=go-dev
http://go/go-review/1018023
|
|
R=gri
http://go/go-review/1015011
|