summaryrefslogtreecommitdiff
path: root/test/bugs
Commit message (Collapse)AuthorAgeFilesLines
* cmd/gc: fix wrong interaction between inlining and embedded builtins.R?my Oudompheng2012-12-223-60/+0
| | | | | | | | | | | | | | | | The patch makes the compile user an ordinary package-local symbol for the name of embedded fields of builtin type. This is incompatible with the fix delivered for issue 2687 (revision 3c060add43fb) but fixes it in a different way, because the explicit symbol on the field makes the typechecker able to find it in lookdot. Fixes issue 3552. R=lvd, rsc, daniel.morsing CC=golang-dev https://codereview.appspot.com/6866047
* test: expand run.go's errorcheck, make clear which bugs runRuss Cox2012-09-232-0/+6
| | | | | | | | | | | | | | | | Today, if run.go doesn't understand a test header line it just ignores the test, making it too easy to write or edit tests that are not actually being run. - expand errorcheck to accept flags, so that bounds.go and escape*.go can run. - create a whitelist of skippable tests in run.go; skipping others is an error. - mark all skipped tests at top of file. Update issue 4139. R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/6549054
* cmd/6g, cmd/8g: eliminate short integer arithmetic when possible.R?my Oudompheng2012-09-011-21/+0
| | | | | | | | | Fixes issue 3909. Fixes issue 3910. R=rsc, nigeltao CC=golang-dev http://codereview.appspot.com/6442114
* exp/locale/collate: avoid 16-bit mathRuss Cox2012-05-241-0/+21
| | | | | | | | | | | There's no need for the 16-bit arithmetic here, and it tickles a long-standing compiler bug. Fix the exp code not to use 16-bit math and create an explicit test for the compiler bug. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/6256048
* cmd/gc: test for issue 3552Luuk van Dijk2012-05-023-0/+57
| | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/6128051
* cmd/gc: do not confuse unexported methods of same nameRuss Cox2012-03-073-86/+0
| | | | | | | | Fixes issue 3146. R=ken2 CC=golang-dev http://codereview.appspot.com/5756074
* test: bug424: wrong embedded method calledRobert Griesemer2012-02-273-0/+86
| | | | | | R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5695083
* gc: undo most of 'fix infinite recursion for embedded interfaces'Russ Cox2012-01-201-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | Preserve test. changeset: 11593:f1deaf35e1d1 user: Luuk van Dijk <lvd@golang.org> date: Tue Jan 17 10:00:57 2012 +0100 summary: gc: fix infinite recursion for embedded interfaces This is causing 'interface type loop' errors during compilation of a complex program. I don't understand what's happening well enough to boil it down to a simple test case, but undoing this change fixes the problem. The change being undone is fixing a corner case (uses of pointer to interface in an interface definition) that basically only comes up in erroneous Go programs. Let's not try to fix this again until after Go 1. Unfixes issue 1909. TBR=lvd CC=golang-dev http://codereview.appspot.com/5555063
* gc: 0 expected bugsRuss Cox2011-12-093-35/+0
| | | | | | | | | Now that Luuk's qualified exporting code is in, fixing this bug is trivial. R=ken2 CC=golang-dev http://codereview.appspot.com/5479048
* test: Add test for inheriting private method from anonymous field.Ian Lance Taylor2011-09-143-0/+35
| | | | | | | | | | | The spec says that all methods are inherited from an anonymous field. There is no exception for non-exported methods. This is related to issue 1536. R=rsc CC=golang-dev http://codereview.appspot.com/5012043
* gc: relax assignability of method receiversAnthony Martin2011-05-243-70/+0
| | | | | | | | | | | | | | | | | The spec was adjusted in commit df410d6a4842 to allow the implicit assignment of strutures with unexported fields in method receivers. This change updates the compiler. Also moved bug322 into fixedbugs and updated golden.out to reflect the removal of the last known bug. Fixes issue 1402. R=golang-dev, gri, rsc CC=golang-dev http://codereview.appspot.com/4526069 Committer: Russ Cox <rsc@golang.org>
* gc: correct handling of unexported method names in embedded interfacesRuss Cox2011-04-213-71/+0
| | | | | | | | | | | | | | go/types: update for export data format change reflect: require package qualifiers to match during interface check runtime: require package qualifiers to match during interface check test: fixed bug324, adapt to be silent Fixes issue 1550. Issue 1536 remains open. R=gri, ken2, r CC=golang-dev http://codereview.appspot.com/4442071
* test: adjust bug324 to expect run-time failure, not compile-time.Ian Lance Taylor2011-03-292-2/+2
| | | | | | | | | | | | | | | Failing at compile time requires that for each conversion between two interface types the compiler compare the sets of unexported methods to see if they come from different packages. Since this test will fail approximately never on real code, and since it can't catch all cases of the problem, I don't think it's worth testing in the compiler. This CL changes this test to look for a run-time panic rather than a compile-time error. R=gri, rsc1, iant2, rsc CC=golang-dev http://codereview.appspot.com/4332041
* test: adjust bugs/bug322 to match current spec.Ian Lance Taylor2011-03-291-5/+5
| | | | | | R=gri, rsc1 CC=golang-dev http://codereview.appspot.com/4328041
* bug324.go: test case for issue 1550Robert Griesemer2011-02-243-0/+71
| | | | | | | | | Also: renamed fixedbugs/bug322.go to fixedbugs/bug323.go because we already have a bugs/bug322.go and bug322.dir. R=rsc CC=golang-dev http://codereview.appspot.com/4219044
* issue 1402: added test caseRobert Griesemer2011-02-033-0/+70
| | | | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/4079063 Committer: Robert Griesemer <gri@golang.org>
* gc: align structs according to max alignment of fieldsRuss Cox2010-12-131-55/+0
| | | | | | | | | | | | | cc: same runtime: test cc alignment (required moving #define of offsetof to runtime.h) fix bug260 Fixes issue 482. Fixes issue 609. R=ken2, r CC=golang-dev http://codereview.appspot.com/3563042
* fix buildRobert Griesemer2010-09-151-24/+0
| | | | | | R=iant CC=golang-dev, ken2 http://codereview.appspot.com/2182043
* test: Add test for //line (currently fails).Ian Lance Taylor2010-09-101-0/+24
| | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/2127045
* gc: bug299, bug300Russ Cox2010-08-032-56/+0
| | | | | | R=ken2 CC=golang-dev http://codereview.appspot.com/1731057
* bug300: literal types must not be parenthesizedRobert Griesemer2010-07-291-0/+29
| | | | | | | | | ( This CL is dependent on acceptance of http://codereview.appspot.com/1913041/show ) R=go-dev CC=golang-dev http://codereview.appspot.com/1860045
* bug299: parenthesized receiver types/anonymous fields are illegalRobert Griesemer2010-07-291-0/+27
| | | | | | R=r CC=golang-dev http://codereview.appspot.com/1846051
* gc: bug274Russ Cox2010-07-151-29/+0
| | | | | | R=ken2 CC=golang-dev http://codereview.appspot.com/1742044
* gc: fix handling of types inside function bodiesRuss Cox2010-07-151-94/+0
| | | | | | | | | Fixes issue 849. Fixes issue 920. R=ken2 CC=golang-dev http://codereview.appspot.com/1841042
* bug286: test caseRobert Griesemer2010-06-091-0/+94
| | | | | | R=r CC=golang-dev http://codereview.appspot.com/1644042
* gc: new typechecking rulesRuss Cox2010-06-082-307/+0
| | | | | | | | | | | | | | | | | * Code for assignment, conversions now mirrors spec. * Changed some snprint -> smprint. * Renamed runtime functions to separate interface conversions from type assertions: convT2I, assertI2T, etc. * Correct checking of \U sequences. Fixes issue 840. Fixes issue 830. Fixes issue 778. R=ken2 CC=golang-dev http://codereview.appspot.com/1303042
* fix buildRuss Cox2010-06-081-1/+1
| | | | | | R=gri CC=golang-dev http://codereview.appspot.com/1616041
* bug284: make conversion legal, fix error patternsRuss Cox2010-06-081-6/+10
| | | | | | R=gri CC=golang-dev http://codereview.appspot.com/1612041
* bug285: delete invalid map assignmentRuss Cox2010-06-081-3/+1
| | | | | | | | *T2 implements I2, but T2 does not. R=gri CC=golang-dev http://codereview.appspot.com/1613041
* bug285: assignment compatible map keys must be acceptedRobert Griesemer2010-06-031-0/+118
| | | | | | R=r CC=golang-dev http://codereview.appspot.com/1473042
* bug284: additional conversion test caseRobert Griesemer2010-06-021-0/+5
| | | | | | R=r, iant CC=golang-dev http://codereview.appspot.com/1433042
* bug284: test cases for new conversion rulesRobert Griesemer2010-06-021-0/+182
| | | | | | R=rsc, r, iant CC=golang-dev http://codereview.appspot.com/1465041
* gc: bug277 - new conversion syntaxRuss Cox2010-05-241-72/+0
| | | | | | R=ken2 CC=golang-dev http://codereview.appspot.com/1220046
* bug277: tests for conversion syntaxRobert Griesemer2010-05-241-0/+72
| | | | | | | | | | - accepted by gccgo, gofmt - some not accepted by 6g - spec revision forthcoming R=rsc CC=golang-dev http://codereview.appspot.com/1279041
* clean up bug 275Ken Thompson2010-05-201-20/+0
| | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/1224044
* bug275: cannot apply real() to a complex array elementRobert Griesemer2010-05-191-0/+20
| | | | | | R=r, ken2 CC=golang-dev http://codereview.appspot.com/1260041
* bug274: test case for issue 777Robert Griesemer2010-05-131-0/+29
| | | | | | R=r, iant CC=golang-dev http://codereview.appspot.com/1195042
* gc: bug264Russ Cox2010-05-031-49/+0
| | | | | | | | Fixes issue 692. R=ken2 CC=golang-dev http://codereview.appspot.com/1092041
* gc: bug267Russ Cox2010-05-031-22/+0
| | | | | | R=ken2 CC=golang-dev http://codereview.appspot.com/1067042
* 6g bug: no need for parens around array index expressionRobert Griesemer2010-04-271-0/+22
| | | | | | | | where index is a composite literal R=rsc CC=golang-dev http://codereview.appspot.com/961044
* gc: bug265Russ Cox2010-03-291-22/+0
| | | | | | | | Fixes issue 700. R=ken2 CC=golang-dev http://codereview.appspot.com/839041
* bug265: test case for issue 700Robert Griesemer2010-03-291-0/+22
| | | | | | | | ( http://code.google.com/p/go/issues/detail?id=700 ) R=r CC=golang-dev http://codereview.appspot.com/827042
* bug264 didn't report BUG correctly, caused "fail" from test/runRob Pike2010-03-251-4/+4
| | | | | | R=rsc, gri CC=golang-dev http://codereview.appspot.com/762041
* bug264: test case for issue 692Robert Griesemer2010-03-251-0/+49
| | | | | | R=r CC=golang-dev http://codereview.appspot.com/715042
* Generate same output on all architectures.Ian Lance Taylor2010-03-051-6/+20
| | | | | | | | | Passing an argument to bug260 will indicate which alignments are wrong. R=r CC=golang-dev http://codereview.appspot.com/231042
* add a test that structures pack. 6g fails.Rob Pike2010-03-041-0/+41
| | | | | | R=rsc CC=golang-dev http://codereview.appspot.com/237041
* gc: recursive interface embeddingRuss Cox2010-02-182-40/+0
| | | | | | | | Fixes issue 287. R=ken2 CC=golang-dev http://codereview.appspot.com/215048
* gc: undo attempt at fixing recursive interface embeddingRuss Cox2010-02-162-0/+40
| | | | | | | | | | | | Fixes issue 582. Update issue 287 Status: Accepted Bug fix was too intrusive; undo and reopen issue. R=ken2 CC=golang-dev http://codereview.appspot.com/209044
* gc: bug246Russ Cox2010-02-011-18/+0
| | | | | | R=ken2 CC=golang-dev http://codereview.appspot.com/198057
* gc: bug242Russ Cox2010-02-011-121/+0
| | | | | | R=ken2 CC=golang-dev http://codereview.appspot.com/198053
View Lorry Controller Status