Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
(caused certain files to not be idempotent under gofmt)
- corrected golden files for go/printer
- slightly simplified some code in nodes.go (no impact on formatting)
- these changes have no impact on gofmt output of .go files under src, misc
fallthrough statement considered harmful!
R=rsc
CC=golang-dev
http://codereview.appspot.com/1593042
|
|
apply gofmt to src, misc
Fixes issue 752.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1240044
Committer: Robert Griesemer <gri@golang.org>
|
|
literal
within an if, for, or switch control clause
R=rsc
CC=golang-dev
http://codereview.appspot.com/943046
|
|
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
|
|
- gofmt -w src misc
- only manually modified file: src/pkg/go/printer/nodes.go
R=rsc
CC=golang-dev, r
http://codereview.appspot.com/606041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/223071
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/223044
|
|
- always include position information about opening/closing parens/braces
- replace uses of []*ast.Field with *ast.FieldList
Fixes issue 473.
R=rsc
CC=golang-dev
http://codereview.appspot.com/223043
|
|
(this will simplify some further changes)
removed several TODOs
R=rsc
CC=golang-dev
http://codereview.appspot.com/216059
|
|
- go/ast: removed StringList (not needed anymore)
- go/ast: changed import path and field list tag to a single string
- updated all dependencies
R=rsc
CC=golang-dev
http://codereview.appspot.com/217056
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/197043
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/194126
|
|
Seems to be enabled spuriously during godoc (can't see why),
producing errors like:
parser.parseDir: src/pkg/http/server.go:159:16: 'Write' declared already at src/pkg/http/request.go:140:21 (and 4 more errors)
R=r
CC=golang-dev
http://codereview.appspot.com/194119
|
|
- provide scope to parse functions; if non-nil, parser uses the scope
to declare and lookup identifiers
- resolve forward references where possible
R=rsc
CC=golang-dev
http://codereview.appspot.com/194098
|
|
Update gofmt test script.
R=rsc
CC=golang-dev
http://codereview.appspot.com/195044
|
|
Fixes issue 567.
R=rsc
CC=golang-dev
http://codereview.appspot.com/195041
|
|
minor cleanups (which I did at home before
but missed this morning at work)
R=rsc
CC=golang-dev
http://codereview.appspot.com/193095
|
|
Fixes issue 530.
R=rsc
CC=golang-dev
http://codereview.appspot.com/193091
|
|
- separate parsing from declaration
- setup of correct scopes
R=rsc
CC=golang-dev
http://codereview.appspot.com/189098
|
|
Wrong position information for identifier
could lead to destructive reformatting of
source via hg gofmt.
R=rsc
CC=golang-dev
http://codereview.appspot.com/189100
|
|
- Identifiers refer now to the language entity (Object)
that they denote. At the moment this is at best an
approximation.
- Initial data structures for language entities (Objects)
and expression types (Type) independent of the actual
type notations.
- Initial support for declaring and looking up identifiers.
- Updated various dependent files and added support functions.
- Extensively tested to avoid breakage. This is an AST change.
R=rsc
CC=golang-dev, rog
http://codereview.appspot.com/189080
|
|
R=agl, agl1
CC=golang-dev
http://codereview.appspot.com/181080
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/179099
|
|
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
3rd set of files.
R=rsc
CC=golang-dev
http://codereview.appspot.com/180048
|
|
R=rsc
http://codereview.appspot.com/175046
Committer: Robert Griesemer <gri@golang.org>
|
|
R=rsc
http://codereview.appspot.com/172041
|
|
Fixes issue 352.
R=rsc
http://codereview.appspot.com/161073
|
|
R=r, r1
http://codereview.appspot.com/157167
|
|
- 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
|
|
R=r, rsc
http://codereview.appspot.com/157082
|
|
rsc's algorithm
- applied gofmt -w misc src
- partial CL (remaining files in other CLs)
R=rsc, r
http://go/go-review/1026036
|
|
R=rsc, r
http://go/go-review/1025029
|
|
- 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
|
|
- added a list of issues to printer/nodes.go
R=rsc
http://go/go-review/1024002
|
|
(previously not shown in godoc)
- simplify parsing of struct types (match code structure for parsing interface types)
R=rsc, r
http://go/go-review/1016019
|
|
- added extra tests
R=rsc
DELTA=55 (37 added, 0 deleted, 18 changed)
OCL=35250
CL=35276
|
|
(in sync with spec CL 35047)
R=rsc
DELTA=44 (4 added, 8 deleted, 32 changed)
OCL=35048
CL=35054
|
|
R=r
OCL=34731
CL=34731
|