summaryrefslogtreecommitdiff
path: root/src/reflect/all_test.go
Commit message (Collapse)AuthorAgeFilesLines
* [dev.cc] all: merge default (8d42099cdc23) into dev.ccdev.ccRuss Cox2014-12-051-0/+101
|\ | | | | | | | | | | TBR=austin CC=golang-codereviews https://codereview.appspot.com/178700044
| * reflect: Fix reflect.funcLayout. The GC bitmap has two bits perKeith Randall2014-12-011-0/+101
| | | | | | | | | | | | | | | | | | | | | | pointer, not one. Fixes issue 9179 LGTM=iant, rsc R=golang-codereviews, iant, rsc CC=golang-codereviews https://codereview.appspot.com/182160043
* | [dev.power64] all: merge default into dev.power64Austin Clements2014-10-221-0/+5
|\ \ | |/ |/| | | | | | | | | | | | | | | | | This brings dev.power64 up-to-date with the current tip of default. go_bootstrap is still panicking with a bad defer when initializing the runtime (even on amd64). LGTM=rsc R=rsc CC=golang-codereviews https://codereview.appspot.com/152570049
| * build: merge the great pkg/ rename into dev.power64Austin Clements2014-10-221-0/+5
| | | | | | | | | | | | | | | | | | This also removes pkg/runtime/traceback_lr.c, which was ported to Go in an earlier commit and then moved to runtime/traceback.go. Reviewer: rsc@golang.org rsc: LGTM
* | reflect: fix TestAllocations now that interfaces hold only pointersIan Lance Taylor2014-10-201-3/+14
| | | | | | | | | | | | | | | | | | | | This test was failing but did not break the build because it was not run when -test.short was used. LGTM=bradfitz R=golang-codereviews, bradfitz CC=golang-codereviews https://codereview.appspot.com/157150043
* | reflect: fix struct size calculation to include terminal paddingDamien Neil2014-10-161-0/+20
| | | | | | | | | | | | | | | | | | LGTM=r R=r CC=golang-codereviews https://codereview.appspot.com/160920045 Committer: Rob Pike <r@golang.org>
* | reflect: generated unrolled GC bitmask directlyIan Lance Taylor2014-10-131-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code for a generated type is already generating an unrolled GC bitmask. Rather than unrolling the the source type bitmasks and copying them, just generate the required bitmask directly. Don't mark it as an unrolled GC program, since there is no need to do so. Fixes issue 8917. LGTM=rsc R=dvyukov, rsc CC=golang-codereviews https://codereview.appspot.com/156930044
* | reflect: add direct call tests to TestMakeFuncVariadicMichael Hudson-Doyle2014-10-081-1/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | TestMakeFuncVariadic only called the variadic function via Call and CallSlice, not via a direct function call. I thought these tests would fail under gccgo tip, but they don't. Still seems worth having though. LGTM=iant R=golang-codereviews, gobot, iant CC=golang-codereviews https://codereview.appspot.com/152060043 Committer: Ian Lance Taylor <iant@golang.org>
* | reflect: add tests for variadic method callsIan Lance Taylor2014-10-081-5/+49
| | | | | | | | | | | | | | | | | | | | | | | | These tests fail when using gccgo. In gccgo using Interface on the value of a method function is implemented using a variant of MakeFunc. That approach did not correctly handle variadic functions. LGTM=r R=golang-codereviews, r CC=golang-codereviews https://codereview.appspot.com/151280043
* | reflect: fix IsValid vs Kind mismatch after Elem of nil interfaceRuss Cox2014-10-011-0/+14
| | | | | | | | | | | | | | LGTM=r R=r CC=golang-codereviews https://codereview.appspot.com/151960044
* | reflect: adjust Value.String to give correct answer for methodsRuss Cox2014-09-181-0/+16
| | | | | | | | | | | | | | | | | | Fixes issue 7859. LGTM=r R=adonovan, r CC=golang-codereviews https://codereview.appspot.com/136710043
* | reflect: add Type.ComparableRuss Cox2014-09-161-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | Like most of the Type methods, the definition of Comparable is what the Go spec says it is. Fixes issue 7911. LGTM=gri R=gri, r CC=golang-codereviews https://codereview.appspot.com/144020043
* | runtime: look up arg stackmap for makeFuncStub/methodValueStub during tracebackRuss Cox2014-09-121-0/+25
|/ | | | | | | | | | | | | | | | | | | | | | | | makeFuncStub and methodValueStub are used by reflect as generic function implementations. Each call might have different arguments. Extract those arguments from the closure data instead of assuming it is the same each time. Because the argument map is now being extracted from the function itself, we don't need the special cases in reflect.Call anymore, so delete those. Fixes an occasional crash seen when stack copying does not update makeFuncStub's arguments correctly. Will also help make it safe to require stack maps in the garbage collector. Derived from CL 142000044 by khr. LGTM=khr R=khr CC=golang-codereviews https://codereview.appspot.com/143890044
* build: move package sources from src/pkg to srcRuss Cox2014-09-081-0/+3862
Preparation was in CL 134570043. This CL contains only the effect of 'hg mv src/pkg/* src'. For more about the move, see golang.org/s/go14nopkg.
View Lorry Controller Status