Robert Hencke
b2329e997b
cgo: fix incorrect print found by govet
...
R=golang-dev
CC=golang-dev
https://golang.org/cl/5445052
2011-11-29 14:40:34 +11:00
Alex Brainman
fbdec642a9
runtime: make sure windows/amd64 stack is 16-byte aligned on syscall entry (fixes build)
...
R=golang-dev, vcc.163
CC=golang-dev
https://golang.org/cl/5445051
2011-11-29 12:57:20 +11:00
Andrew Balholm
c32b607687
html: detect quirks mode
...
Pass tests3.dat, test 23:
<p><table></table>
| <html>
| <head>
| <body>
| <p>
| <table>
R=nigeltao
CC=golang-dev
https://golang.org/cl/5446043
2011-11-29 11:18:49 +11:00
Alex Brainman
6d4c18a4af
syscall: implement Syscall15
...
Fixes #2251 .
R=golang-dev, rsc
CC=golang-dev, jp
https://golang.org/cl/5440050
2011-11-29 10:24:19 +11:00
Andrew Gerrand
5a18aef67c
goinstall: add -fix flag to run gofix on packages on build failure
...
goinstall: better error handling and reporting
R=r, r, rsc, mattn.jp
CC=golang-dev
https://golang.org/cl/5421051
2011-11-29 09:28:58 +11:00
Russ Cox
8e515485e2
gc: remove funarg special case in structfield
...
This should make CL 5431046 a little simpler.
R=ken2
CC=golang-dev
https://golang.org/cl/5444048
2011-11-28 16:40:39 -05:00
Charles L. Dorian
c8d2544b26
math: update special-conditions comments to use ± symbol
...
R=rsc, golang-dev, r
CC=golang-dev
https://golang.org/cl/5445046
2011-11-28 13:04:52 -08:00
Dave Cheney
4cc64bd5bf
exp/ssh: move openChan to NewSession
...
openChan was only being called by NewSession, Dial has
its own version.
R=gustav.paul, agl, rsc
CC=golang-dev
https://golang.org/cl/5435071
2011-11-28 15:42:47 -05:00
Adam Langley
1eb7ca924b
crypto/tls: don't rely on map iteration order.
...
Previously we were using the map iteration order to set the order of
the cipher suites in the ClientHello.
R=bradfitz
CC=golang-dev
https://golang.org/cl/5440048
2011-11-28 15:34:16 -05:00
Rob Pike
5f6027e9ad
text/template: address a couple of issues for html/template
...
- allow Lookup to work on uninitialized templates
- fix bug in add: can't error after parser is stopped
- add Add method for html/template
R=adg, rogpeppe, r, rsc
CC=golang-dev
https://golang.org/cl/5436080
2011-11-28 10:42:57 -08:00
Christopher Wedgwood
356b8ee26f
strings: fix test output
...
R=rsc, gri
CC=golang-dev
https://golang.org/cl/5445044
2011-11-28 09:51:03 -08:00
Christopher Wedgwood
7600281bda
bytes: fix test output
...
R=rsc, gri
CC=golang-dev
https://golang.org/cl/5441048
2011-11-28 09:50:51 -08:00
Christopher Wedgwood
62203141ac
doc/tmptohtml: output fix
...
R=r, gri
CC=golang-dev
https://golang.org/cl/5441047
2011-11-28 09:50:40 -08:00
Dave Cheney
b57bb9282e
exp/ssh: add safeString error sanitiser
...
R=huin, agl, gustav.paul, cw
CC=golang-dev
https://golang.org/cl/5399044
2011-11-28 12:29:19 -05:00
Rémy Oudompheng
60e4a61d30
gc: don't inherit orig from subnodes in constant expression nodes.
...
The wrong value made Nconv() show "1" for node "-1", and "2" from
node "2+3".
Fixes #2452 .
R=gri, lvd, rsc
CC=golang-dev, remy
https://golang.org/cl/5435064
2011-11-28 12:22:15 -05:00
Dave Cheney
ce7e11997b
exp/ssh: fix three shift bugs related to packet lengths
...
Thanks for Ke Lan for the initial report and investigation.
R=agl, gustav.paul, tg8866, rsc
CC=golang-dev
https://golang.org/cl/5443044
2011-11-28 12:10:16 -05:00
Maxim Pimenov
ffa6b383f5
gc: fix copying of types
...
reset xmethod during copytype
Fixes #2497
R=rsc, dvyukov
CC=golang-dev
https://golang.org/cl/5441045
2011-11-28 11:52:16 -05:00
Brad Fitzpatrick
1e85f41fd5
http: fix sniffing bug causing short writes
...
R=rsc
CC=golang-dev
https://golang.org/cl/5442045
2011-11-28 11:51:34 -05:00
Russ Cox
7606079d9f
CONTRIBUTORS: add Maxim Pimenov (Google CLA)
...
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5443047
2011-11-28 11:47:23 -05:00
Brad Fitzpatrick
e77099daa2
sql: add Tx.Stmt to use an existing prepared stmt in a transaction
...
R=rsc
CC=golang-dev
https://golang.org/cl/5433059
2011-11-28 11:00:32 -05:00
Gustav Paul
23227f3d63
exp/ssh: fix misleading spelling mistake in comment
...
R=dave, agl, rsc, golang-dev
CC=golang-dev
https://golang.org/cl/5431089
2011-11-28 10:45:52 -05:00
Gustavo Niemeyer
a620865639
filepath/path: fix Rel buffer sizing
...
Fixes #2493 .
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5433079
2011-11-27 21:28:52 -05:00
Charles L. Dorian
f1fecf8d2a
math: fix typo in Log1p comments
...
Also note the special case of -0.0.
R=rsc, golang-dev, gri
CC=golang-dev
https://golang.org/cl/5435084
2011-11-27 18:01:08 -08:00
Andrew Balholm
68e7363b56
html: parse <nobr> elements
...
Pass tests3.dat, test 20:
<!doctype html><nobr><nobr><nobr>
| <!DOCTYPE html>
| <html>
| <head>
| <body>
| <nobr>
| <nobr>
| <nobr>
Also pass tests through test 22:
<!doctype html><html><body><p><table></table></body></html>
R=nigeltao
CC=golang-dev
https://golang.org/cl/5438056
2011-11-28 10:55:31 +11:00
Michael T. Jones
4c113ffe16
math/big: use recursive subdivision for significant speedup
...
This change adds the second aspect to the conversion code, the
use of large divisiors (powers of big base) to greatly speed up
the divsion of large numbers. Speedups of 30x are common in the
large cases. Also includes new tests and tuning code for the
key internal parameters.
R=gri
CC=golang-dev
https://golang.org/cl/5438058
2011-11-27 11:10:59 -08:00
Gustav Paul
d859d7deee
exp/ssh: messages now contain remote channel's id instead of local id
...
According to http://www.ietf.org/rfc/rfc4254.txt most channel messages contain the channel id of the recipient channel, not the sender id. This allows the recipient connection multiplexer to route the message to the correct channel.
This changeset fixes several messages that incorrectly send the local channel id instead of the remote channel's id.
While sessions were being created and closed in sequence channels in the channel pool were freed and reused on the server side of the connection at the same rate as was done on the client, so the channel local and remote channel ids always corresponded. As soon as I had concurrent sessions on the same clientConn the server started to complain of 'uknown channel id N' where N is the local channel id, which is actually paired with server channel id K.
R=golang-dev, dave, rsc, agl
CC=golang-dev
https://golang.org/cl/5433063
2011-11-27 09:59:20 -05:00
Andrew Balholm
557ba72e69
html: ignore <head> tags in <head> element
...
Pass tests3.dat, test 12:
<!DOCTYPE html><HTML><META><HEAD></HEAD></HTML>
| <!DOCTYPE html>
| <html>
| <head>
| <meta>
| <body>
Also pass tests through test 19:
<!DOCTYPE html><html><head></head><body><ul><li><div><p><li></ul></body></html>
R=nigeltao
CC=golang-dev
https://golang.org/cl/5436069
2011-11-27 14:41:08 +11:00
Rob Pike
e9025df7ad
text/template: rename the method Template.Template to Template.Lookup
...
Calling it Template makes it clumsy to embed the type, which html/template
depends on.
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5432079
2011-11-26 08:32:55 -08:00
Alex Brainman
e38a1053a9
os: fail if Open("") is called on windows
...
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5432071
2011-11-26 11:01:49 +11:00
Benny Siegert
5519b5d716
exp/sql/driver: Correct package name in comment.
...
IIRC, package sql used to be called db. There was one occurrence
of the old name in a comment.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5431075
2011-11-25 10:57:10 -05:00
Brad Fitzpatrick
1f83c2b6d5
C+A: add Gustav Paul (individual CLA)
...
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5432076
2011-11-25 10:50:19 -05:00
Dmitriy Vyukov
a899a467f2
time: fix timer stop
...
Due to data structure corruption,
some timers could not be removed.
Fixes #2495 .
R=golang-dev, adg
CC=golang-dev, mdbrown
https://golang.org/cl/5437060
2011-11-25 14:13:10 +03:00
Andrew Gerrand
38c082f69e
html/template: fix documentation indent
...
R=nigeltao
CC=golang-dev
https://golang.org/cl/5437061
2011-11-25 13:32:44 +11:00
Andrew Gerrand
49dfaad870
dashboard: builder-facing implementation and tests
...
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5431048
2011-11-25 12:53:05 +11:00
Rob Pike
0197cc49ae
text/template: fix bug in Clone
...
Cloned template copied the root template incorrectly.
Add test of self-consistency.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5436063
2011-11-24 16:07:19 -08:00
Rob Pike
d1324d8a7a
effective_go: remove unused variable from a couple of examples
...
Fixes #2481 .
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5435068
2011-11-24 08:51:47 -08:00
Scott Lawrence
0f0c25dccc
strings: Add ContainsAny and ContainsRune to correspond to IndexAny etc.
...
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5430046
2011-11-23 20:20:14 -08:00
Rob Pike
da62104169
gofix: trivial rewrite for template.ParseFiles
...
Also warn about uses of Set.
R=rsc
CC=golang-dev
https://golang.org/cl/5436051
2011-11-23 20:17:41 -08:00
Rob Pike
f56db6f534
text/template: new, simpler API
...
The Set type is gone. Instead, templates are automatically associated by
being parsed together; nested definitions implicitly create associations.
Only associated templates can invoke one another.
This approach dramatically reduces the breadth of the construction API.
For now, html/template is deleted from src/pkg/Makefile, so this can
be checked in. Nothing in the tree depends on it. It will be updated next.
R=dsymonds, adg, rsc, r, gri, mikesamuel, nigeltao
CC=golang-dev
https://golang.org/cl/5415060
2011-11-23 20:17:22 -08:00
Andrew Balholm
af081cd43e
html: ingore newline at the start of a <pre> block
...
Pass tests3.dat, test 4:
<!DOCTYPE html><html><head></head><body><pre>\n</pre></body></html>
| <!DOCTYPE html>
| <html>
| <head>
| <body>
| <pre>
Also pass tests through test 11:
<!DOCTYPE html><pre>

A</pre>
R=nigeltao
CC=golang-dev
https://golang.org/cl/5437051
2011-11-24 13:15:09 +11:00
Robert Griesemer
63e48ccd8e
go/ast: trivial cleanup (remove superfluous string conversion)
...
R=iant, bradfitz
CC=golang-dev
https://golang.org/cl/5430059
2011-11-23 16:20:55 -08:00
Andrew Balholm
77b0ad1e80
html: parse DOCTYPE into name and public and system identifiers
...
Pass tests2.dat, test 59:
<!DOCTYPE <!DOCTYPE HTML>><!--<!--x-->-->
| <!DOCTYPE <!doctype>
| <html>
| <head>
| <body>
| ">"
| <!-- <!--x -->
| "-->"
Pass all the tests in doctype01.dat.
Also pass tests2.dat, test 60:
<!doctype html><div><form></form><div></div></div>
R=nigeltao
CC=golang-dev
https://golang.org/cl/5437045
2011-11-24 09:28:58 +11:00
Robert Griesemer
b3923a27dd
go/printer, gofmt: more performance tweaks
...
Removed more string conversions and streamlined bottleneck
printing interface by removing unnecessary tests where possible.
About 6% faster AST printing.
Before:
- printer.BenchmarkPrint 50 32056640 ns/op
After:
- printer.BenchmarkPrint 50 30138440 ns/op (-6%)
R=r
CC=golang-dev
https://golang.org/cl/5431047
2011-11-23 09:27:38 -08:00
Rémy Oudompheng
8362ee99b0
fmt: don't check for nil when printing arrays as Go syntax.
...
Also add array values to printing test suite.
Fixes #2468 .
R=golang-dev, r
CC=golang-dev, remy
https://golang.org/cl/5436053
2011-11-23 09:04:02 -08:00
Adam Langley
8281f6bd1b
crypto/openpgp/packet: fix private key checksum
...
I misinterpreted http://tools.ietf.org/html/rfc4880#section-5.5.3
and implemented the sum of 16-bit values, rather than the 16-bit sum
of 8-bit values.
Thanks to Szabolcs Nagy for pointing it out.
R=bradfitz, r, rsc
CC=golang-dev
https://golang.org/cl/5372091
2011-11-23 09:44:29 -05:00
Dave Cheney
f2c858749a
exp/ssh: add simple tcpip.go functional tests
...
R=rsc, agl, n13m3y3r, bradfitz
CC=golang-dev
https://golang.org/cl/5385041
2011-11-23 09:38:11 -05:00
Dave Cheney
d5514120b1
exp/ssh: remove unused forwarding methods in Server Listener
...
R=agl, rsc
CC=golang-dev
https://golang.org/cl/5436056
2011-11-23 09:35:01 -05:00
Robert Griesemer
4874d14180
go/printer: remove "written" result value - is never used
...
R=r
CC=golang-dev
https://golang.org/cl/5436052
2011-11-22 15:27:10 -08:00
Robert Griesemer
a0e54aaffa
go/printer, gofmt: 20 to 30% faster gofmt
...
Buffer intermediate output via a bytes.Buffer and thus avoid
calling through the entire Writer stack for every item printed.
There is more opportunity for improvements along the same lines.
Before (best of 3 runs):
- printer.BenchmarkPrint 50 47959760 ns/op
- time gofmt -l $GOROOT/src real 0m11.517s
After (best of 3 runs):
- printer.BenchmarkPrint 50 32056640 ns/op (= -33%)
- time gofmt -l $GOROOT/src real 0m9.070s (= -21%)
R=r
CC=golang-dev
https://golang.org/cl/5432054
2011-11-22 15:12:34 -08:00
Andrew Balholm
57ed39fd3b
html: on EOF in a comment, ignore final dashes (up to 2)
...
Pass tests2.dat, test 57:
<!DOCTYPE html><!--x--
| <!DOCTYPE html>
| <!-- x -->
| <html>
| <head>
| <body>
Also pass test 58:
<!DOCTYPE html><table><tr><td></p></table>
R=nigeltao
CC=golang-dev
https://golang.org/cl/5436048
2011-11-23 09:26:37 +11:00