fmt format verb %b bug

fmt.Printf("%b", int8(-1)) prints 64 ones instead of 8.
This happens only for signed integers (int8, in16 and int32). I guess it's because of the way the conversion between integer types works. From go spec: "Conversions between integer types. If the value is a signed quantity, it is sign extended to implicit infinite precision ....". And there are several conversions to int64 and uint64 in the fmt package. This pathch solves only half of the problem. On a 32 bit system, an fmt.Printf("%b", int(-1)) should still print 64 ones.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/891049

CLA: Andrei Vieru

R=rsc
CC=golang-dev
https://golang.org/cl/890044

json: update documentation to match current coding style

R=rsc
CC=golang-dev
https://golang.org/cl/888045

gc: compile s == "" as len(s) == 0

R=ken2
CC=golang-dev
https://golang.org/cl/840043

gc: distinguish fatal compiler bug from error+exit

R=ken2
CC=golang-dev
https://golang.org/cl/902044

gc: make sure main.main has correct type

R=ken2
CC=golang-dev
https://golang.org/cl/883049

debug/elf: Fix doc comments

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/849049

exp/eval: Fix example and add target to Makefile

R=rsc
CC=golang-dev
https://golang.org/cl/901042

archive/tar: update documentation to match current coding style

R=rsc, r
CC=golang-dev
https://golang.org/cl/903044

nacl: update documentation to match current coding style

R=rsc, r
CC=golang-dev
https://golang.org/cl/822047

testing: update documentation to match current coding style

R=rsc, r
CC=golang-dev
https://golang.org/cl/823045

godoc: change od.Dir -> os.FileInfo in comments

R=gri, r
CC=golang-dev, rsc
https://golang.org/cl/819042

time: fix comment typo

R=r
CC=golang-dev
https://golang.org/cl/811044

codereview: Mention that deleting a CL is a use of 'hg change' in its header.

R=rsc
CC=golang-dev
https://golang.org/cl/903043

strings: add IndexRune, Trim, TrimLeft, TrimRight, and the generic equivalents TrimFunc, TrimLeftFunc, TrimRightFunc

R=rsc, r
CC=golang-dev
https://golang.org/cl/799048

runtime: delete malx, skip_depth argument to malloc

remove internal functions from traces in gopprof instead.

R=r
CC=golang-dev
https://golang.org/cl/855046

freebsd: fix build, maybe

R=r
CC=golang-dev
https://golang.org/cl/909041

math: use ** for exponentiation in comments

R=rsc
CC=golang-dev
https://golang.org/cl/908041

cmath: use ** for exponentiation in comments

R=rsc
CC=golang-dev
https://golang.org/cl/831045

Run initcgo for all amd64 targets, not just GNU/Linux.

This is required to make cgo export work on Darwin.  Note that
this corrects the stack alignment when calling initcgo to that
required by gcc on amd64.

R=rsc
CC=golang-dev
https://golang.org/cl/907041

A test case for cgo //export.

R=rsc
CC=golang-dev
https://golang.org/cl/881043

Add //export to cgo.

The new //export comment marks a Go function as callable from
C.  The syntax is "//export NAME" where NAME is the name of
the function as seen from C.  If such a comment is seen, cgo
will generate two new files: _cgo_export.h and _cgo_export.c.
The _cgo_export.h file provides declarations which C code may
use to call Go functions.  The _cgo_export.c file contains
wrappers, and is to be compiled with gcc.

The changes to Make.pkg support using this from a Go Makefile,
though it could probably be more convenient.

R=rsc
CC=golang-dev
https://golang.org/cl/853042

Library support for cgo export.

These functions are used to call from a C function back to a
Go function.  This only includes 386 support.

R=rsc
CC=golang-dev
https://golang.org/cl/834045

Support cgo export on amd64.

R=rsc
CC=golang-dev
https://golang.org/cl/857045

Update roadmap: delete many accomplishments

R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/854045

rename os.Dir to os.FileInfo

R=rsc
CC=golang-dev
https://golang.org/cl/902042

fix bug in os_test.go checkMode

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/872043

runtime: fix build (panic) for FreeBSD

R=adg
CC=golang-dev
https://golang.org/cl/867046

runtime: turn divide by zero, nil dereference into panics

tested on linux/amd64, linux/386, linux/arm, darwin/amd64, darwin/386.
freebsd untested; will finish in a separate CL.

for now all the panics are errorStrings.
richer structures can be added as necessary
once the mechanism is shaked out.

R=r
CC=golang-dev
https://golang.org/cl/906041

runtime: fix bad status throw
when garbage collector sees recovering goroutine

Fixes #711.

R=r
CC=golang-dev
https://golang.org/cl/869045

runtime: work around kernel bug in Snow Leopard signal handling

Could not take a signal on threads other than the main thread.
If you look at the spinning binary with dtrace, you can see a
fault happening over and over:

    $ dtrace -n '
    fbt::user_trap:entry /execname=="boot32" && self->count < 10/
    {
        self->count++;
        printf("%s %x %x %x %x", probefunc, arg1, arg2, arg3, arg4);
        stack();
        tracemem(arg4, 256);
    }'

    dtrace: description 'fbt::user_trap:entry ' matched 1 probe
    CPU     ID                    FUNCTION:NAME
      1  17015                  user_trap:entry user_trap 0 10 79af0a0 79af0a0
                  mach_kernel`lo_alltraps+0x12a

             0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f  0123456789abcdef
         0: 0e 00 00 00 37 00 00 00 00 00 00 00 1f 00 00 00  ....7...........
        10: 1f 00 00 00 a8 33 00 00 00 00 00 01 00 00 00 00  .....3..........
        20: 98 ba dc fe 07 09 00 00 00 00 00 00 98 ba dc fe  ................
        30: 06 00 00 00 0d 00 00 00 34 00 00 00 9e 1c 00 00  ........4.......
        40: 17 00 00 00 00 02 00 00 ac 30 00 00 1f 00 00 00  .........0......
        50: 00 00 00 00 00 00 00 00 0d 00 00 00 e0 e6 29 00  ..............).
        60: 34 00 00 00 00 00 00 00 9e 1c 00 00 00 00 00 00  4...............
        70: 17 00 00 00 00 00 00 00 00 02 00 00 00 00 00 00  ................
        80: ac 30 00 00 00 00 00 00 1f 00 00 00 00 00 00 00  .0..............
        90: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
        a0: 48 00 00 00 10 00 00 00 85 00 00 00 a0 f2 29 00  H.............).
        b0: 69 01 00 02 00 00 00 00 e6 93 04 82 ff 7f 00 00  i...............
        c0: 2f 00 00 00 00 00 00 00 06 02 00 00 00 00 00 00  /...............
        d0: 78 ee 42 01 01 00 00 00 1f 00 00 00 00 00 00 00  x.B.............
        e0: 00 ed 9a 07 00 00 00 00 00 00 00 00 00 00 00 00  ................
        f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................

    ...

The memory dump shows a 32-bit exception frame:

    x86_saved_state32

    gs = 0x37
    fs = 0
    es = 0x1f
    ds = 0x1f
    edi = 0x33a8
    esi = 0x01000000
    ebp = 0
    cr2 = 0xfedcba98
    ebx = 0x0907
    edx = 0
    ecx = 0xfedcba98
    eax = 0x06
    trapno = 0x0d
    err = 0x34
    eip = 0x1c9e
    cs = 0x17
    efl = 0x0200
    uesp = 0x30ac
    ss = 0x1f

The cr2 of 0xfedcba98 is the address that the new thread read
to cause the fault, but note that the trap is now a GP fault with
error code 0x34, meaning it's moved past the cr2 problem and on
to an invaild segment selector.  The 0x34 is suspiciously similar
to the 0x37 in gs, and sure enough, OS X forces gs to have
that value in the signal handler, and if your thread hasn't set
up that segment (known as USER_CTHREAD), you'll fault on the IRET
into the signal handler and never be able to handle a signal.

The kernel bug is that it forces segment 0x37 without making sure
it is a valid segment.  Leopard also forced 0x37 but had the courtesy
to set it up first.

Since OS X requires us to set up that segment (using the
thread_fast_set_cthread_self system call), we might as well
use it instead of the more complicated i386_set_ldt call to
set up our per-OS thread storage.

Also add some more zeros to bsdthread_register for new arguments
in Snow Leopard (apparently unnecessary, but being careful).

Fixes #510.

R=r
CC=golang-dev
https://golang.org/cl/824046

math: atan2 special cases (negative zero)

Added Signbit(), revised Copysign()

R=rsc
CC=golang-dev
https://golang.org/cl/822045

runtime: use explicit flag when finalizer goroutine is waiting

Avoids spurious wakeups during other sleeping by that goroutine.
Fixes #711.

R=r
CC=golang-dev
https://golang.org/cl/902041

gotest: update URL printed by failure message.
Fixes #677.

R=rsc
CC=golang-dev
https://golang.org/cl/834046

gp_spec typo fix: code example in the "Handling panics" section

R=rsc, adg
CC=golang-dev
https://golang.org/cl/881047

Language FAQ: editing tweaks after iant.

R=iant
CC=golang-dev
https://golang.org/cl/874043

Language FAQ: update the entry on exceptions.

R=rsc, iant
CC=golang-dev
https://golang.org/cl/824045

arm: fix build, attempt #2

TBR=kaib
CC=golang-dev
https://golang.org/cl/897041

arm: fix build, attempt #1

TBR=kaib
CC=golang-dev
https://golang.org/cl/883046

another try at clearing
channel recv data.

R=rsc
CC=golang-dev
https://golang.org/cl/896041

change channel read to clear
data just read from the channel.
this will make it easier to
recognize when to garbage
collect and finalize.

R=rsc
CC=golang-dev
https://golang.org/cl/882043

net: use chan bool instead of chan *netFD to avoid cycle

The cycle is *netFD -> cw chanl *netFD in struct ->
same *netFD in channel read buffer.

Because channels are finalized, the cycle makes them
uncollectable.  A better fix is to make channels not
finalized anymore, and that will happen, but this is
an easy, reasonable workaround until then.

Another good fix would be to zero the channel receive
buffer entry after the receive.  That too will happen.

R=r
CC=golang-dev
https://golang.org/cl/875043

flags: better tests.

R=rsc
CC=golang-dev
https://golang.org/cl/864044

runtime: two proc bug fixes

1. Fix bug in GOMAXPROCS when trying to cut number of procs
Race could happen on any system but was
manifesting only on Xen hosted Linux.

2. Fix recover on ARM, where FP != caller SP.

R=r
CC=golang-dev
https://golang.org/cl/880043

Change goyacc to be reentrant.
Instead of calling the package scope Lex function,
Parse now takes an argument which is used to
do the lexing.
I reverted to having the generated switch
code inside Parse rather than a separate function because
the function needs 7 arguments or a context structure,
which seems unnecessary.
I used yyrun(), not the original $A so that
it's possible to run the backquoted code through gofmt.

R=rsc, ken2, ken3
CC=golang-dev
https://golang.org/cl/879041

replace original float instruction with jump to make branches
to float instructions work correctly.

R=rsc
CC=golang-dev
https://golang.org/cl/870044

fix deps.bash. \t does not mean tab in some seds.
also: add /dev/null to the ls args to repair handling of empty dependency lists.

R=rsc
CC=golang-dev
https://golang.org/cl/883045

syscall package: document that errno is zeroed on success

This is a documentation enhancement only, without any code
change.

The rationale for documenting this precisely is that Unix
programmers who "know" that errno's value is undefined after
a successful system call may be surprised otherwise and
search to be sure that a zero errno may be relied upon after
successful calls.

R=r, rsc1, rsc
CC=golang-dev
https://golang.org/cl/812044

GNU/Linux sed requires \? rather than ?.  Just use * instead.

R=rsc, r
CC=golang-dev
https://golang.org/cl/810043

libcgo: initial mingw port work - builds but untested

R=rsc
CC=golang-dev
https://golang.org/cl/812041

io/ioutil: fix bug in ReadFile when Open succeeds but Stat fails

R=gri
CC=golang-dev
https://golang.org/cl/867044

runtime: fix Caller
log: add test of Caller

New regexp in log test is picky and will require some
maintenance, but it catches off-by-one mistakes too.

Fixes #710.

R=gri
CC=esko.luontola, golang-dev
https://golang.org/cl/887043

http: fix documentation example

R=adg
CC=golang-dev
https://golang.org/cl/813043

test/bench: add k-nucleotide-parallel

R=rsc
CC=golang-dev
https://golang.org/cl/881042

cmath: new package

Complex math function package. Still needs more special case checking.

R=rsc
CC=golang-dev
https://golang.org/cl/874041

deps.bash: be less strict about format of x.go lines

R=iant
CC=golang-dev
https://golang.org/cl/862042

doc/go_mem: remove semicolons

R=adg
CC=golang-dev
https://golang.org/cl/893041

runtime: handle malloc > 2GB correctly

R=ken2
CC=golang-dev
https://golang.org/cl/821048

crypto/tls: good defaults

R=agl1
CC=golang-dev
https://golang.org/cl/851041

runtime: various arm fixes
  * correct symbol table size
  * do not reorder functions in output
  * traceback
  * signal handling
  * use same code for go + defer
  * handle leaf functions in symbol table

R=kaib, dpx
CC=golang-dev
https://golang.org/cl/884041

programming_faq: added question on T vs *T method sets

Adding this question on Russ' recommendation - not sure if
there is some detail here I'm missing.

The associated discussion was:
http://groups.google.com/group/golang-nuts/t/ec6b27e332ed7f77

R=rsc, r
CC=golang-dev
https://golang.org/cl/887042

gc: good syntax error for defer func() {} - missing final ()

R=ken2
CC=golang-dev
https://golang.org/cl/855044

nntp: new package, NNTP client

R=rsc, rsc1
CC=golang-dev
https://golang.org/cl/808041

run.bash: remove MAKEFLAGS=-j4 to prevent crashing on freebsd

R=rsc
CC=golang-dev
https://golang.org/cl/881044

godashboard: fix benchmarks page (first pass)

R=rsc
CC=golang-dev
https://golang.org/cl/844044

debug/proc: fix typo in package documentation

R=rsc
CC=golang-dev
https://golang.org/cl/829044

syscall: implementing some mingw syscalls required by os package

R=rsc
CC=golang-dev
https://golang.org/cl/770041

runtime: turn run time errors checks into panics

R=ken2, r
CC=golang-dev
https://golang.org/cl/871042

debug/macho: fix error message format

R=rsc
CC=golang-dev
https://golang.org/cl/836046

debug/macho: don't crash when reading non-Mach-O files

R=rsc
CC=golang-dev
https://golang.org/cl/838046

go spec: correct clarification of type declaration

R=rsc, r
CC=golang-dev
https://golang.org/cl/855043

runtime: correct memory leak in select
  * adds pass 3 to dequeue from channels eagerly

various other cleanup/churn:
  * use switch on cas->send in each pass to
    factor out common code.
  * longer goto labels, commented at target
  * be more agressive about can't happen:
    throw instead of print + cope.
  * use "select" instead of "selectgo" in errors
  * use printf for debug prints when possible

R=ken2, ken3
CC=golang-dev, r
https://golang.org/cl/875041

kate: Update for recent language changes

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/849045

build script tweaks

factor out environment variable checks.
infer $GOROOT etc during build if not set.
it's still necessary to set them for yourself
to use the standard Makefiles.

when running all.bash, don't recompile all the
go packages in run.bash, since make.bash already did.

R=r
CC=golang-dev
https://golang.org/cl/609042

testing/regexp: use recover.

R=rsc
CC=golang-dev
https://golang.org/cl/816042

json: use panic/recover to handle errors in Marshal

R=r, gri
CC=golang-dev
https://golang.org/cl/872041

go spec: clarification of type declarations

R=r, rsc
CC=golang-dev
https://golang.org/cl/849044

Delete xgb from the main repository. It has moved to
http://code.google.com/p/x-go-binding/

R=rsc
CC=golang-dev
https://golang.org/cl/846043

gc: fix alignment on non-amd64

R=ken2
CC=golang-dev
https://golang.org/cl/870041

tabwriter: use panic/recover to handle errors

R=rsc, r
CC=golang-dev
https://golang.org/cl/864042

regexp: use panic/recover to handle errors

R=rsc, gri
CC=golang-dev
https://golang.org/cl/821046

runtime: make type assertion a runtime.Error, the first of many

R=r
CC=golang-dev
https://golang.org/cl/805043

test for panic and recover

R=r, adg
CC=golang-dev
https://golang.org/cl/869041

gc: implement panic and recover

R=ken2, r, ken3
CC=golang-dev
https://golang.org/cl/831042

godashboard: new stylesheet, build status pagination

R=rsc, gri, r
CC=golang-dev
https://golang.org/cl/822044

release.2010-03-30 part two

R=rsc
CC=golang-dev
https://golang.org/cl/859042

Unicode: fix stupid typo in comment.

R=rsc
CC=golang-dev
https://golang.org/cl/864041

Unicode: provide an ability to supplement the case-mapping tables
in character and string case mapping routines.

Add a custom mapper for Turkish and Azeri.

A more general solution for deriving the case information from Unicode's
SpecialCasing.txt will require more work.

Fixes #703.

R=rsc, rsc1
CC=golang-dev, mdakin
https://golang.org/cl/824043

release 2010-03-30

R=rsc, gri
CC=golang-dev
https://golang.org/cl/821045

godoc: support for title and subtitle headers when serving .html docs
and use it to show version (date) of go spec

Fixes #68.

R=rsc
CC=golang-dev, r
https://golang.org/cl/848042

runtime: fix arm build, slightly.

R=r
CC=golang-dev
https://golang.org/cl/842042

go/printer: follow-up on CL 802043
- more test cases
- comment fixes
- minor unrelated changes as part of investigation of issue 702

R=rsc
CC=golang-dev
https://golang.org/cl/860041

time: do not segment time strings by character class.
instead use pure substring matching to find template values.

this makes stdZulu unnecessary and allows formats
like "20060102 030405" (used in some internet protocols).

this makes Parse not handle years < 0000 or > 9999 anymore.
that seems like an okay price to pay, trading hypothetical
functionality for real functionality.

also changed the comments on the Time struct to use the
same reference date as the format and parse routines.

R=r
CC=golang-dev
https://golang.org/cl/833045

syscall: add IPV6 constants

R=r
CC=golang-dev
https://golang.org/cl/848041

single argument panic on non-darwin and in comments

R=r
CC=golang-dev
https://golang.org/cl/800042

Gofmt preserves newlines in multiline selector expressions.

This is for making the fluent interface idiom usable with gofmt.

R=gri
CC=golang-dev
https://golang.org/cl/802043

contributors, authors: add Risto Saarelma

R=rsc
CC=golang-dev
https://golang.org/cl/836043

Effective Go: update maps description regarding lookup of nonexistent entry.

R=rsc, gri, iant
CC=golang-dev
https://golang.org/cl/821044

godoc: don't print package clause in -src command-line mode with filtering

R=rsc
CC=golang-dev
https://golang.org/cl/844041

gc: add panic and recover (still unimplemented in runtime)
main semantic change is to enforce single argument to panic.

runtime: change to 1-argument panic.
use String method on argument if it has one.

R=ken2, r
CC=golang-dev
https://golang.org/cl/812043