bug209.go

R=rsc
DELTA=23  (22 added, 1 deleted, 0 changed)
OCL=35529
CL=35537

Move rand.go to rng.go to prepare for some surgery.

R=rsc
APPROVED=rsc
DELTA=637  (318 added, 318 deleted, 1 changed)
OCL=35534
CL=35536

time tests; sort -nr times.out | sed 10q is illuminating.
cut the slowest tests  down from a few seconds
to under half a second.

R=r
DELTA=21  (6 added, 1 deleted, 14 changed)
OCL=35509
CL=35519

fix nacl build

R=r
DELTA=23  (5 added, 0 deleted, 18 changed)
OCL=35510
CL=35518

floating point calc, added a few more packages that pass

go/test: passes 80% (274/340)

R=rsc
APPROVED=rsc
DELTA=61  (59 added, 0 deleted, 2 changed)
OCL=35506
CL=35517

rewrite errchk in perl for speed (compared to bash)

R=iant
DELTA=125  (51 added, 53 deleted, 21 changed)
OCL=35508
CL=35511

add & fix bug208, from ken.
fix bug198.

R=ken
OCL=35504
CL=35507

dupok, gcc compile fix, sync and syscall asm fix

R=rsc
APPROVED=rsc
DELTA=27  (18 added, 0 deleted, 9 changed)
OCL=35503
CL=35505

- debugging support

R=rsc
DELTA=110  (98 added, 0 deleted, 12 changed)
OCL=35487
CL=35490

more lgtm files from gofmt

R=gri
OCL=35485
CL=35488

Only clean the relevant installed files, not everything.  This
should let a 6g build and an 8g build be done in sequence
without stepping on each other.

R=rsc
DELTA=2  (0 added, 0 deleted, 2 changed)
OCL=35478
CL=35481

- rewrite declaration printing to take full use of discardable tabwriter columns
- honor line breaks in multi-line expressions
- do not add extra indentation to multi-line string lists
- don't put blanks around simple function calls and conversions
- do not modify `` strings
- added extra test cases

R=rsc
DELTA=398  (246 added, 51 deleted, 101 changed)
OCL=35453
CL=35465

PNG encoder unit test.

R=rsc,r
APPROVED=r
DELTA=84  (77 added, 6 deleted, 1 changed)
OCL=35456
CL=35458

add & fix bug207: rewritten if condition
was discarding initialization work.

R=ken
OCL=35454
CL=35457

better handling of mistaken top-level variable
references during the parsing of :=.  the base
problem is that when reading

a,b,c,d

the parser makes those refer to existing variables,
which might create a few stub top-level ones
for undefined names, but then if a := is the next
token, we need to undo those stubs.

this was causing problems in multifile packages
in which one file used a := variable named rpc
and the other imported a package named rpc.

R=ken
OCL=35446
CL=35446

do not import testing twice when testing testing.

R=r
DELTA=3  (2 added, 0 deleted, 1 changed)
OCL=35443
CL=35445

apply gofmt to rand reflect regexp rpc runtime sort strconv strings sync syscall testing time unicode unsafe utf8

R=gri
DELTA=1409  (79 added, 24 deleted, 1306 changed)
OCL=35415
CL=35437

support for "hard" and "soft" tabs:
- soft-tab separated columns can be discarded if empty and DiscardEmptyColumns is set
- hard-tab separated columns are never discarded

R=rsc
DELTA=63  (42 added, 7 deleted, 14 changed)
OCL=35421
CL=35435

factor portable object+library bits out of 5l/6l/8l into ld

R=r
DELTA=3214  (904 added, 2260 deleted, 50 changed)
OCL=35425
CL=35427

make reader more useful
for lower-level clients:
* expose p.Skip
* expose p.Unmarshal
* wildcard struct field "Any"
* unmarshal into bool
* unmarshal into xml.Name
* unmarshal into pointer

R=r
DELTA=61  (50 added, 5 deleted, 6 changed)
OCL=35372
CL=35422

apply gofmt to go, gob, hash, http, image, io, json, log

R=gri
DELTA=1359  (138 added, 32 deleted, 1189 changed)
OCL=35408
CL=35420

apply gofmt to malloc math once os

R=gri
DELTA=566  (4 added, 14 deleted, 548 changed)
OCL=35410
CL=35419

apply gofmt to auto-generated syscall files

R=gri
DELTA=4023  (104 added, 27 deleted, 3892 changed)
OCL=35416
CL=35418

5l library loading, 64 bit entry points

R=rsc
APPROVED=rsc
DELTA=58  (52 added, 3 deleted, 3 changed)
OCL=35417
CL=35417

- moved 3 functions from syscall_linux to _386 and _amd64 (arm lacks them)
- 64 bit OMINUS
- added bunch of missing arm syscalls

R=rsc
APPROVED=rsc
DELTA=203  (189 added, 3 deleted, 11 changed)
OCL=35412
CL=35414

fix build - missing from 35404

TBR=gri
OCL=35411
CL=35411

- set uint type for flags
- simplified some code

R=rsc
DELTA=19  (0 added, 0 deleted, 19 changed)
OCL=35405
CL=35407

apply gofmt to datafmt, ebnf, exec, expvar, flag, fmt

R=gri
DELTA=456  (6 added, 3 deleted, 447 changed)
OCL=35398
CL=35406

more comment work.
got rid of regexps.
primary bug fix is that // inside /* */ do not get stripped anymore,
so that the text inside

/*
int a;
// int b;
int c;
*/

is

int a;
// int b;
int c;

before, the "int b;" line was being uncommented too.

R=gri
DELTA=65  (13 added, 42 deleted, 10 changed)
OCL=35334
CL=35404

- simplify "needsBlanks" logic for identifiers and strings

TBR=rsc
DELTA=16  (10 added, 4 deleted, 2 changed)
OCL=35379
CL=35403

add IgnoreEmptyColumns feature to tabwriter

R=rsc
DELTA=96  (74 added, 2 deleted, 20 changed)
OCL=35391
CL=35402

fix build

R=rsc
DELTA=5  (5 added, 0 deleted, 0 changed)
OCL=35387
CL=35401

another round of gofmt applications

R=gri
DELTA=900  (106 added, 31 deleted, 763 changed)
OCL=35384
CL=35396

gofmt on crypto, debug

R=gri
DELTA=2560  (127 added, 177 deleted, 2256 changed)
OCL=35388
CL=35395

cleaned up cgen() to be a bit more straightforward.

R=rsc
APPROVED=rsc
DELTA=104  (46 added, 56 deleted, 2 changed)
OCL=35392
CL=35394

apply gofmt to the LGTM-marked files from 34501
that have not changed since I applied gofmt.

R=gri
DELTA=456  (77 added, 3 deleted, 376 changed)
OCL=35378
CL=35383

Change description of time.Time.ZoneOffset to specify east is +ve.

Currently, the description says that +ve numbers are westwards of UTC.
Typically, timezones are specified with +ve numbers running east of Greenwich.
For example, San Francisco is specified as UTC-8. Also, when calling
time.Localtime(), the following is returned on my box in PDT:

{Year:2009 Month:10 Day:5 Hour:18 Minute:55 Second:6 Weekday:1 ZoneOffset:-25200
Zone:PDT}

Note that the ZoneOffset is negative. This leads me to suspect that the
description is mistaken.

R=rsc
APPROVED=rsc
DELTA=1  (0 added, 0 deleted, 1 changed)
OCL=35356
CL=35377

change tabwidth to 8 for tests

R=rsc
DELTA=39  (0 added, 0 deleted, 39 changed)
OCL=35360
CL=35376

64 bit cmp and some sgen tweaks

go/test: passes 75% (256/339)

R=rsc
APPROVED=rsc
DELTA=142  (53 added, 4 deleted, 85 changed)
OCL=35367
CL=35375

move cas out, add dummy runcgo.

R=rsc
APPROVED=rsc
DELTA=73  (45 added, 28 deleted, 0 changed)
OCL=35363
CL=35366

fix arm test to actual return success/fail and expose runcgo
build breakage.

R=rsc
APPROVED=rsc
DELTA=10  (4 added, 2 deleted, 4 changed)
OCL=35362
CL=35365

allow multiple -L options

R=rsc
DELTA=31  (15 added, 3 deleted, 13 changed)
OCL=35364
CL=35364

various go printer fixes:
- better handling of line breaks in expression lists
- fixed line breaks around label decls
- remove ()'s around if, for, switch expressions
- simple index expressions don't require blanks
- better line breaks around declarations of different kind

R=rsc
DELTA=404  (369 added, 8 deleted, 27 changed)
OCL=35354
CL=35359

remove a file from idempotency test so that the entire suite passes

R=rsc
DELTA=2  (0 added, 0 deleted, 2 changed)
OCL=35355
CL=35358

First cut at a PNG encoder.
TODOs include filtering, and a unit test.

R=rsc
APPROVED=r
DELTA=280  (249 added, 1 deleted, 30 changed)
OCL=35262
CL=35348

comment tweaks; implement precise name
character sets.

R=r
DELTA=339  (257 added, 1 deleted, 81 changed)
OCL=35344
CL=35346

add a little more saturation to the background of <pre> so it doesn't disappear on bright monitors.
verified as acceptable on my calibrated home monitor.

R=gri
DELTA=1  (0 added, 0 deleted, 1 changed)
OCL=35340
CL=35342

XML parser

R=r
DELTA=546  (545 added, 0 deleted, 1 changed)
OCL=35318
CL=35341

XML lexing

The lexer is the bottom level.
Most clients will use the Unmarshal method,
not yet implemented, which will behave like
json.Unmarshal.

R=r
DELTA=1115  (766 added, 219 deleted, 130 changed)
OCL=35316
CL=35339

addressing gri's comments.

R=gri
DELTA=31  (7 added, 17 deleted, 7 changed)
OCL=35336
CL=35338

slices
methods on slices, pointers and values.

R=iant,rsc,gri
DELTA=173  (150 added, 1 deleted, 22 changed)
OCL=35333
CL=35337

preserve blank lines in // comments

R=gri
DELTA=32  (13 added, 12 deleted, 7 changed)
OCL=35317
CL=35332

update json comments

R=r
DELTA=16  (4 added, 2 deleted, 10 changed)
OCL=35320
CL=35331

a few fixups. delete the TODO block.

R=rsc
DELTA=23  (0 added, 20 deleted, 3 changed)
OCL=35330
CL=35330

Add write support for the GNU tar binary numeric field extension.

R=rsc
APPROVED=rsc
DELTA=102  (89 added, 1 deleted, 12 changed)
OCL=35321
CL=35327

fix build

TBR=r
OCL=35308
CL=35308

add cgo test that doesn't depend on
non-standard libraries and add to build.

R=r
DELTA=211  (210 added, 0 deleted, 1 changed)
OCL=35293
CL=35307

Fixing HTTP POST handling to work with Chrome and Safari.

request.go does not handle Content-Type correctly for the definition of
Media Types.

http://www.w3.org/Protocols/rfc2616/rfc2616-sec3.html#sec3.7

R=rsc
APPROVED=rsc
DELTA=44  (42 added, 0 deleted, 2 changed)
OCL=35274
CL=35306

clean more

R=r
DELTA=40  (9 added, 3 deleted, 28 changed)
OCL=35277
CL=35305

8c, 8l dynamic loading support.
better mach binaries.
cgo working on darwin+linux amd64+386.
eliminated context switches - pi is 30x faster.
add libcgo to build.

on snow leopard:
  - non-cgo binaries work; all tests pass.
  - cgo binaries work on amd64 but not 386.

R=r
DELTA=2031  (1316 added, 626 deleted, 89 changed)
OCL=35264
CL=35304

expand error regexp in dns test

R=r
DELTA=3  (0 added, 0 deleted, 3 changed)
OCL=35292
CL=35303

garbage collections, conversions, concurrency

R=iant,rsc,gri
DELTA=106  (87 added, 8 deleted, 11 changed)
OCL=35298
CL=35302

- improved handling of white space around declarations and statements
- extra tests

R=rsc
DELTA=366  (264 added, 37 deleted, 65 changed)
OCL=35299
CL=35301

add comment about C API.
somehow managed not to Put this before.

R=r
DELTA=3  (3 added, 0 deleted, 0 changed)
OCL=35289
CL=35294

final Native Client package - av

R=r
DELTA=894  (887 added, 0 deleted, 7 changed)
OCL=35115
CL=35286

- allow parenthesized [...]T types as in: ([...]int){}
- added extra tests

R=rsc
DELTA=55  (37 added, 0 deleted, 18 changed)
OCL=35250
CL=35276

a few more code generation bugs and an interface alignment issue.

go/test: passes 74% (251/339)

R=rsc
APPROVED=rsc
DELTA=40  (34 added, 0 deleted, 6 changed)
OCL=35254
CL=35275

Change deflate.go's default compression level from 4 to 6.

R=rsc
APPROVED=rsc
DELTA=1  (0 added, 0 deleted, 1 changed)
OCL=35208
CL=35261

replace "ideal" with "untyped"

R=r
DELTA=1  (0 added, 0 deleted, 1 changed)
OCL=35242
CL=35258

use the notion of "untyped constant" instead of "ideal constant"

R=iant
DELTA=13  (1 added, 0 deleted, 12 changed)
OCL=35241
CL=35246

Factored out boilerplate from all html docs in doc directory:
- the first HTML comment in those files is extracted as page
  title when serving them
- lib/godoc.html is top-level template for all pages served
- experimented a bit with package documentation layout
  (feedback welcome)
- various related cleanups

TODO:
- The / page (doc/root.html) content repeats links that are
  in the navigation bar. It needs to be cleaned up.

R=rsc
DELTA=826  (86 added, 692 deleted, 48 changed)
OCL=35230
CL=35245

generate float registers correctly.

go/test: passes 71% (242/339)

R=rsc
APPROVED=rsc
DELTA=9  (5 added, 0 deleted, 4 changed)
OCL=35233
CL=35243

some progress on arm linux syscall interface. ztypes_linux_arm
still has problems with godefs.

R=rsc
APPROVED=rsc
DELTA=801  (134 added, 235 deleted, 432 changed)
OCL=35189
CL=35211

cgen_asop sudoaddable

R=rsc
APPROVED=rsc
DELTA=14  (4 added, 0 deleted, 10 changed)
OCL=35190
CL=35210

fixed a few calls to gcmp and some really bothched OINDEX code
(how did that happen?)

go/test: passes 70% (238/339)

R=rsc
APPROVED=rsc
DELTA=18  (10 added, 0 deleted, 8 changed)
OCL=35185
CL=35188

step toward darwin ffi: factor out macho code

R=r
DELTA=771  (374 added, 287 deleted, 110 changed)
OCL=35187
CL=35187

load binary op arguments into registers

go/test: passes 69% (237/339)

R=rsc
APPROVED=rsc
DELTA=12  (3 added, 9 deleted, 0 changed)
OCL=35178
CL=35180

cgo working on linux/386

R=r
DELTA=70  (47 added, 4 deleted, 19 changed)
OCL=35167
CL=35171

rename the public exvar package to be expvar.

R=rsc
DELTA=684  (324 added, 324 deleted, 36 changed)
OCL=35161
CL=35163

- completed section on built-in functions
- moved Conversions section out of built-in functions and into expressions
- fixed syntax of conversions (parens are not mandatory if the type is not a TypeName)
  (this is the only change to the Conversions section; the rest of the text is just moved;
  old line: 4043, new line: 3078)
- fixed syntax of composite literals (parens are allowed around LiteralType)

DELTA=239  (115 added, 98 deleted, 26 changed)
OCL=35118
CL=35159

cgo: works on amd64.
integrated into Makefiles (see misc/cgo/gmp/Makefile).

R=r
DELTA=1110  (540 added, 525 deleted, 45 changed)
OCL=35153
CL=35158

delete

R=r
DELTA=791  (0 added, 791 deleted, 0 changed)
OCL=35154
CL=35154

c compiler bug tickled by
void f(struct { int x[1]; } p) { }
the "int" was getting attached to f.

R=ken
OCL=35145
CL=35151

another brace of questions: types, maps, bookkeeping

DELTA=136  (120 added, 13 deleted, 3 changed)
OCL=35133
CL=35147

clean up error output;
do not print entire buffer on error,
since it can be very large.
instead print identifying string.

R=krasin
DELTA=14  (1 added, 3 deleted, 10 changed)
OCL=35141
CL=35143

fix 386 build - register allocation and shift bug

R=ken
OCL=35140
CL=35142

missing error checking related to ([...]int){...}

R=ken
OCL=35132
CL=35132

nacl system call updates

R=r
DELTA=236  (211 added, 18 deleted, 7 changed)
OCL=35084
CL=35131

undo 35108 (disallow parens around type in struct literal).
allow parens around [...]int in struct literal.

R=ken
OCL=35112
CL=35130

ZLIB deflater.

R=rsc
APPROVED=rsc
DELTA=222  (219 added, 0 deleted, 3 changed)
OCL=35031
CL=35129

Add a Vim syntax highlighting file for Go.

APPROVED=rsc
DELTA=190  (190 added, 0 deleted, 0 changed)
OCL=35073
CL=35128

fix munging of pointer.

go/test: passes 69% (235/339)

R=rsc
APPROVED=rsc
DELTA=5  (4 added, 1 deleted, 0 changed)
OCL=35107
CL=35109

disallow parens around type in struct literal syntax,
per discussion with gri.

R=ken
OCL=35108
CL=35108

Native Client SRPC (simple RPC), both server and client.

R=r
DELTA=958  (958 added, 0 deleted, 0 changed)
OCL=35096
CL=35106

toy draw implementation

R=r
DELTA=471  (471 added, 0 deleted, 0 changed)
OCL=35090
CL=35099

Deflate encoder

APPROVED=rsc
OCL=34514
CL=35093

Add crypto/rc4.

RC4 is a common stream cipher. This adds a pure-go C implementation.

R=r
APPROVED=r
DELTA=139  (138 added, 0 deleted, 1 changed)
OCL=35056
CL=35092

make the sidebars consistent
add Effective Go to the set of links

DELTA=166  (62 added, 57 deleted, 47 changed)
OCL=35085
CL=35089

more language FAQ

DELTA=79  (61 added, 3 deleted, 15 changed)
OCL=35083
CL=35088

fixed alignment issue resulting from confusion around the
meaning of out args.

go/test: passes 68% (231/339)

R=rsc
APPROVED=rsc
DELTA=13  (7 added, 0 deleted, 6 changed)
OCL=35072
CL=35080