- 05 Aug, 2012 5 commits
-
-
Rémy Oudompheng authored
benchmark old ns/op new ns/op delta BenchmarkCompareStringEqual 51 35 -30.20% BenchmarkCompareStringIdentical 51 7 -85.71% BenchmarkCompareStringSameLength 25 18 -28.29% BenchmarkCompareStringDifferentLength 2 2 +1.46% R=golang-dev, rsc CC=golang-dev, remy https://golang.org/cl/6450092
-
Russ Cox authored
This line helps me to tell whether the CL is waiting for me or I'm waiting for the author. Also: - vertical-align table cells so buttons are always aligned with CL headers. - add email= to show front page for someone else. Demo at http://rsc.gocodereview.appspot.com/. Until this is deployed for real, some recently changed CLs may be missing the 'first line of last message' part. R=dsymonds CC=golang-dev https://golang.org/cl/6446065
-
Russ Cox authored
Too many people use it without reading what it does. Those people want ReadBytes or ReadString. Fixes #3906. R=golang-dev, iant, r CC=golang-dev https://golang.org/cl/6442087
-
Rémy Oudompheng authored
The improvement is obtained by eliminating the zero initialization of a large structure that is only needed when the fast path fails. Also add a missing roundtrip test for float32s. benchmark old ns/op new ns/op delta BenchmarkAppendFloatDecimal 301 180 -40.20% BenchmarkAppendFloat 486 388 -20.16% BenchmarkAppendFloatExp 492 383 -22.15% BenchmarkAppendFloatNegExp 478 370 -22.59% BenchmarkAppendFloatBig 650 541 -16.77% BenchmarkAppendFloat32Integer 308 180 -41.56% BenchmarkAppendFloat32ExactFraction 449 333 -25.84% BenchmarkAppendFloat32Point 494 390 -21.05% BenchmarkAppendFloat32Exp 488 387 -20.70% BenchmarkAppendFloat32NegExp 488 378 -22.54% R=r, rsc CC=golang-dev, remy https://golang.org/cl/6346081
-
Andrew Balholm authored
The content of an HTML <title> element is RCDATA, but the content of an SVG <title> element is parsed as tags. Now the parser doesn't go into RCDATA mode in foreign content. Pass 4 additional tests. R=nigeltao CC=golang-dev https://golang.org/cl/6448111
-
- 04 Aug, 2012 3 commits
-
-
Dmitriy Vyukov authored
I have C functions implemented in .syso file (rather than .so or inlined in .go file). W/o this change the gcc invocation fails with undefined symbols. R=minux.ma, rsc CC=golang-dev https://golang.org/cl/6352076
-
Dmitriy Vyukov authored
E.g. sync/atomic.LoadInt32() can be called as sync»atomic·LoadInt32() R=rsc CC=golang-dev https://golang.org/cl/6448057
-
Ian Lance Taylor authored
R=adg, rsc, franciscossouza, seb.binet, gen.battle CC=golang-dev https://golang.org/cl/5845071
-
- 03 Aug, 2012 13 commits
-
-
Michał Derkacz authored
R=golang-dev, dave, rsc, minux.ma CC=golang-dev https://golang.org/cl/6225051
-
Rémy Oudompheng authored
Fixes #3899. R=rsc CC=golang-dev, remy https://golang.org/cl/6453084
-
Rémy Oudompheng authored
The compiler is incorrectly rejecting switches on arrays of comparable types. It also doesn't catch incomparable structs when typechecking the switch, leading to unreadable errors during typechecking of the generated code. Fixes #3894. R=rsc CC=gobot, golang-dev, r, remy https://golang.org/cl/6442074
-
Adam Langley authored
These aren't needed for scalar multiplication, but since we export a generic Add function we should handle it. This change also corrects two bugs in p224Contract that it turned up. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6458076
-
Russ Cox authored
PAX systems are Linux systems that are more paranoid about memory permissions. These flags tell them to relax when running Go binaries. Fixes #47. R=iant CC=golang-dev https://golang.org/cl/6326054
-
Mike Rosset authored
R=rsc CC=golang-dev https://golang.org/cl/6430064
-
Robert Griesemer authored
R=rsc CC=golang-dev https://golang.org/cl/6453083
-
Adam Langley authored
This removes some (non-short only) tests that involved connections to the outside world and thus were flakey. This does remove some test coverage of the root fetchers on non-Windows platforms, but the right place for that is crypto/x509. R=golang-dev, krautz, rsc CC=golang-dev https://golang.org/cl/6455085
-
Adam Langley authored
We already support reading ECDSA certificates and this change adds write support. R=golang-dev, bradfitz, rsc CC=golang-dev https://golang.org/cl/6422046
-
Marcel van Lohuizen authored
for dealing with CLDR files: - Add now taxes a list of indexes of colelems that are variables. Checking and handling is now done by the Builder. VariableTop is now also properly generated using the Build method. - Introduced separate Builder, called Tailoring, for creating tailorings of root table. This clearly separates the functionality for building a table based on weights (the allkeys* files) versus tables based on LDML XML files. - Tailorings are now added by two calls instead of one: SetAnchor and Insert. This more closely reflects the structure of LDML side and simplifies the implementation of both the client and library side. It also preserves some information that is otherwise hard to recover for the Builder. - Allow the LDML XML element extend to be passed to Insert. This simplifies both client and library implementation. R=r CC=golang-dev https://golang.org/cl/6454061
-
Alex Brainman authored
R=golang-dev, r CC=golang-dev https://golang.org/cl/6442080
-
Robert Griesemer authored
R=r CC=golang-dev https://golang.org/cl/6442076
-
Robert Griesemer authored
Don't print unexported struct fields; their values are not accessible via reflection. Fixes #3898. Also: - added support for arrays - print empty maps, arrays, slices, structs on one line for a denser output - added respective test cases R=r CC=golang-dev https://golang.org/cl/6454089
-
- 02 Aug, 2012 3 commits
-
-
Andrew Balholm authored
Pass 2 additional tests. R=nigeltao CC=golang-dev https://golang.org/cl/6454090
-
Nigel Tao authored
R=andybalholm CC=golang-dev https://golang.org/cl/6453071
-
Rémy Oudompheng authored
R=golang-dev, gri CC=golang-dev https://golang.org/cl/6445068
-
- 01 Aug, 2012 7 commits
-
-
Robert Griesemer authored
Process a package's object in a reproducible order (rather then in map order) so that we get error messages in reproducible order. R=r CC=golang-dev https://golang.org/cl/6449076
-
Andrew Balholm authored
The text inside <script> tags is not ordinary raw text; there are all sorts of other complications. This CL implements those complications. Pass 76 additional tests. R=nigeltao CC=golang-dev https://golang.org/cl/6443070
-
Shenghou Ma authored
we can't import "./bug0" on windows, as it will trigger "import path contains invalid character ':'" error. instead, we pass "-D." and "-I." to gc to override this behavior. this idea is due to remyoudompheng. R=golang-dev, r, alex.brainman, remyoudompheng CC=golang-dev https://golang.org/cl/6441074
-
Shenghou Ma authored
Fixes #3866. R=rsc, r, nigeltao CC=golang-dev https://golang.org/cl/6452046
-
Alex Brainman authored
Fixes #3828. R=golang-dev, iant, rsc CC=golang-dev https://golang.org/cl/6420056
-
Robert Griesemer authored
This is more in sync with the rest of the package; for instance, we have functions (not methods) to deref or find the underlying type of a Type. In the process use a single bytes.Buffer to create the string representation for a type rather than the (occasional) string concatenation. R=r CC=golang-dev https://golang.org/cl/6458057
-
Robert Griesemer authored
Also: - replaced existing test with a more comprehensive test - fixed bug in map type creation R=r CC=golang-dev https://golang.org/cl/6450072
-
- 31 Jul, 2012 5 commits
-
-
Andrew Balholm authored
If an end tag has an attribute that is a quoted string containing '>', the tokenizer would end the tag prematurely. Now it reads the attributes on end tags just as it does on start tags, but the high-level interface still doesn't return them, because their presence is a parse error. Pass 1 additional test. R=nigeltao CC=golang-dev https://golang.org/cl/6457060
-
Nigel Tao authored
R=r CC=golang-dev https://golang.org/cl/6446062
-
Rémy Oudompheng authored
The receive operator was given incorrect precedence resulting in incorrect deletion of parentheses. Fixes #3843. R=rsc CC=golang-dev, remy https://golang.org/cl/6442049
-
Mikio Hara authored
uuid_t is defined in unistd.h on OS X, unfortunately. R=iant, rsc CC=golang-dev https://golang.org/cl/6455057
-
Mikio Hara authored
R=rsc CC=golang-dev https://golang.org/cl/6398052
-
- 30 Jul, 2012 4 commits
-
-
Shenghou Ma authored
Android NDK's gcc 4.6 generates this relocation for runtime/cgo. R=rsc CC=golang-dev https://golang.org/cl/6450056
-
Shenghou Ma authored
LLVM-based gcc will place all-zero data in a zero-filled section, but our debug/macho can't handle that. Fixes #3821. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6444049
-
Bobby Powers authored
If the X-Forwarded-For header already exists on a request, we should append our client's IP to it after a comma+space instead of overwriting it. Fixes #3846. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6448053
-
Rob Pike authored
R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6448081
-