- 31 Oct, 2012 12 commits
-
-
Adam Langley authored
Fixes #1390. R=golang-dev, minux.ma, adg, agl CC=golang-dev https://golang.org/cl/6631044
-
Robert Griesemer authored
Fixes #4314. R=r, rsc CC=golang-dev https://golang.org/cl/6822059
-
Rob Pike authored
Fixes #2874. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6820068
-
Rob Pike authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6812060
-
Rob Pike authored
Fixes build, too. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6775088
-
Marcel van Lohuizen authored
R=r, mpvl CC=golang-dev https://golang.org/cl/6818067
-
Marcel van Lohuizen authored
Tailorings are represented by removing and reinserting entries from a linked list. After all tailorings are done, missing weights are computed and verified. This implementation assumes that entries that are used in expansions are not reinserted at a later point. This considerably simplifies the implementation. R=r CC=golang-dev https://golang.org/cl/6739052
-
Marcel van Lohuizen authored
incremental comparisons. Instead, processing is now done directly on colElems. As a result, the size of the weights array is now reduced by 75%. Details: - Primary value of type 1 colElem is shifted by 1 bit so that primaries of all types can be compared without shifting. - Quaternary values are now stored in the colElem itself. This is possible as quaternary values other than 0 or maxQuaternary are only needed when other values are ignored. - Simplified processWeights by removing cases that are needed for ICU but not for us (our CJK primary values fit in a single value). R=r CC=golang-dev https://golang.org/cl/6817054
-
Marcel van Lohuizen authored
R=r CC=golang-dev https://golang.org/cl/6727049
-
Ian Lance Taylor authored
I fixed a bug in gccgo that was causing it to only give an error for the first package that was imported and not used. R=golang-dev, r CC=golang-dev https://golang.org/cl/6813058
-
Peter Waller authored
As discussed in the following thread: https://groups.google.com/forum/?fromgroups=#!topic/golang-dev/emeJffxWhVo This is required to enable applications such as `less` to use something other than stdin as the controlling terminal. R=dave, iant CC=bradfitz, golang-dev https://golang.org/cl/6785057
-
Ian Lance Taylor authored
R=golang-dev, dsymonds CC=golang-dev https://golang.org/cl/6775083
-
- 30 Oct, 2012 14 commits
-
-
Alex Brainman authored
It is common to close network connection while another goroutine is blocked reading on another goroutine. This sequence corresponds to windows calls to WSARecv to start io, followed by GetQueuedCompletionStatus that blocks until io completes, and, finally, closesocket called from another thread. We were expecting that closesocket would unblock GetQueuedCompletionStatus, and it does, but not always (http://code.google.com/p/go/issues/detail?id=4170#c5). Also that sequence results in connection is being reset. This CL inserts CancelIo between GetQueuedCompletionStatus and closesocket, and waits for both WSARecv and GetQueuedCompletionStatus to complete before proceeding to closesocket. This seems to fix both connection resets and issue 4170. It also makes windows code behave similar to unix version. Unfortunately, CancelIo needs to be called on the same thread as WSARecv. So we have to employ strategy we use for connections with deadlines to every connection now. It means, there are 2 unavoidable thread switches for every io. Some newer versions of windows have new CancelIoEx api that doesn't have these drawbacks, and this CL uses this capability when available. As time goes by, we should have less of CancelIo and more of CancelIoEx systems. Computers with CancelIoEx are also not affected by issue 4195 anymore. Fixes #3710 Fixes #3746 Fixes #4170 Partial fix for issue 4195 R=golang-dev, mikioh.mikioh, bradfitz, rsc CC=golang-dev https://golang.org/cl/6604072
-
Nigel Tao authored
benchmark old ns/op new ns/op delta BenchmarkDecodeBaseline 3155638 2783998 -11.78% BenchmarkDecodeProgressive 4008088 3660310 -8.68% R=r, bradfitz CC=golang-dev https://golang.org/cl/6775072
-
Alexey Borzenkov authored
Fixes #4296. R=golang-dev, alex.brainman CC=golang-dev https://golang.org/cl/6815044
-
Rob Pike authored
Keys use ==; values use deep equality. Also remove the word 'member'. Fixes #4258. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6812055
-
Robert Griesemer authored
Remove trailing whitespace in comments. No other changes. R=r CC=golang-dev https://golang.org/cl/6815053
-
Robert Griesemer authored
Also: updated go fix testcases to pass tests. Fixes #4310. R=r CC=golang-dev https://golang.org/cl/6810055
-
Dave Cheney authored
Fixes #4302. R=golang-dev, bradfitz, adg CC=golang-dev https://golang.org/cl/6811048
-
Robert Griesemer authored
R=r CC=golang-dev https://golang.org/cl/6822051
-
Shenghou Ma authored
compiler_rt introduces a weak and hidden symbol compilerrt_abort_impl into our pre-linked _all.o object, we have to handle it. Fixes #4273. R=iant, rsc, r CC=golang-dev https://golang.org/cl/6783050
-
Dave Cheney authored
Occasionally GAE will return a 500 error response, don't treat this as a valid JSON body. R=adg, dsymonds CC=golang-dev https://golang.org/cl/6775066
-
Brad Fitzpatrick authored
Fixes #4303 R=golang-dev, adg CC=golang-dev https://golang.org/cl/6816058
-
Brad Fitzpatrick authored
Feature extraction was tested before, but not the final diffs. This CL breaks function main into a smaller main + testable compareAPI. No functional changes. R=golang-dev, adg CC=golang-dev https://golang.org/cl/6820057
-
Brad Fitzpatrick authored
API change. R=golang-dev, dsymonds, nigeltao, rsc, r CC=golang-dev https://golang.org/cl/6760045
-
Nigel Tao authored
On 6g/linux: benchmark old ns/op new ns/op delta BenchmarkFDCT 4606 4241 -7.92% BenchmarkIDCT 4187 3923 -6.31% BenchmarkDecodeBaseline 3154864 3170224 +0.49% BenchmarkDecodeProgressive 4072812 4017132 -1.37% BenchmarkEncode 39406920 34596760 -12.21% Stack requirements before (from 'go tool 6g -S'): (scan.go:37) TEXT (*decoder).processSOS+0(SB),$1352-32 (writer.go:448) TEXT (*encoder).writeSOS+0(SB),$5344-24 after: (scan.go:37) TEXT (*decoder).processSOS+0(SB),$1064-32 (writer.go:448) TEXT (*encoder).writeSOS+0(SB),$2520-24 Also, in encoder.writeSOS, re-use the yBlock scratch buffer for Cb and Cr. This reduces the stack requirement slightly, but also avoids an unlucky coincidence where a BenchmarkEncode stack split lands between encoder.writeByte and bufio.Writer.WriteByte, which occurs very often during Huffman encoding and is otherwise disasterous for the final benchmark number. FWIW, the yBlock re-use *without* the s/int/int32/ change does not have a noticable effect on the benchmarks. R=r CC=golang-dev, rsc https://golang.org/cl/6823043
-
- 29 Oct, 2012 11 commits
-
-
Roger Peppe authored
"JSON object" means something specific, which isn't the case here. R=golang-dev, r CC=golang-dev https://golang.org/cl/6789044
-
Dmitriy Vyukov authored
Fixes #4306. R=golang-dev, iant CC=golang-dev https://golang.org/cl/6816054
-
Pawel Szczur authored
Fix the problem with no cookie handling when sending other than GET or HEAD request through (*Client) Do(*Request) (*Resposne, error). https://code.google.com/p/go/issues/detail?id=3985 Adds a function (*Client) send(*Request) (*Reponse, error): - sets cookies from CookieJar to request, - sends request - parses a reply cookies and updates CookieJar Fixes #3985 R=bradfitz CC=gobot, golang-dev https://golang.org/cl/6653049
-
Brad Fitzpatrick authored
R=golang-dev, r CC=golang-dev https://golang.org/cl/6820049
-
Adam Langley authored
RFC 5280 section 4.1.2.5.1 says so. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6775068
-
Adam Langley authored
Previously we converted a time to UTC *and* serialized the timezone of the original time. With this change, we serialize a UTCTime in the original timezone. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6817048
-
Luuk van Dijk authored
- make sure dclcontext == PAUTO only in function bodies - introduce PDISCARD to discard declarations in bodies of repeated imports - skip printing initializing OAS'es in export mode, assuming they only occur after ODCL's - remove ODCL and the initializing OAS from inl.c:ishairy - fix confused use of ->typecheck in typecheckinl: it's about the ->inl, not about the fn. - debuging aids: print ntype on ONAMEs too and -Emm instead of -Ell. fixes #2812 R=rsc CC=golang-dev https://golang.org/cl/6800043
-
Luuk van Dijk authored
includes step 0: synthesize outparams, from 6600044 includes step 1,2: give outparams loopdepth 0 and verify unchanged results generate esc:$mask tags, but still tie to sink if a param has mask != 0 from 6610054 adds final steps: - have esccall generate n->escretval, a list of nodes the function results flow to - use these in esccall and ORETURN/OAS2FUNC/and f(g()) - only tie parameters to sink if tag is absent, otherwise according to mask, tie them to escretval R=rsc, bradfitz CC=dave, gobot, golang-dev, iant, rsc https://golang.org/cl/6741044
-
Andrew Radev authored
The "did_ftplugin" lines were ineffective and the "K" mapping was too invasive, which is why it was removed. R=golang-dev, dsymonds, minux.ma CC=golang-dev https://golang.org/cl/6823044
-
David Symonds authored
R=golang-dev, bradfitz CC=andrey.radev, golang-dev https://golang.org/cl/6775067
-
David Symonds authored
Too many people quote entire emails and put their reply at the top ("top posting"), so we shouldn't recognise review commands anywhere in the review text. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6815048
-
- 28 Oct, 2012 2 commits
-
-
Rémy Oudompheng authored
It is essentially identical to the version in 6g. R=dave, minux.ma, rsc CC=golang-dev https://golang.org/cl/6710043
-
Taj Khattra authored
R=golang-dev, fullung, dave, minux.ma CC=golang-dev https://golang.org/cl/6682046
-
- 27 Oct, 2012 1 commit
-
-
Patrick Smith authored
R=golang-dev, dsymonds CC=golang-dev https://golang.org/cl/6777066
-