Robert Griesemer [Thu, 5 Nov 2009 22:03:56 +0000 (14:03 -0800)]
2nd attempt: no noIndent for string lists
- slightly better output in general
- extra indentation where we could do without
(however that seems better for now that not having the indentation
where it is needed)
- no information is lost, so a future better approach can fix these
places up again
Robert Griesemer [Thu, 5 Nov 2009 06:07:13 +0000 (22:07 -0800)]
more comment formatting:
- preserve (some) indentation of comment text for /*-style comments
even if the first comment line does not contain any text that might
suggest the "correct" indentation
- enabled because otherwise existing larger comments get re-formatted
(this will not introduce a lot of changes since comments of this
kind - until now - were not changed with respect to indentation)
Robert Griesemer [Thu, 5 Nov 2009 00:50:13 +0000 (16:50 -0800)]
rename testfiles from .go -> .orig so that they won't be
picked up by gofmt (they should not be modified) or godoc
(testdata should not be considered a potential package
directory)
Russ Cox [Wed, 4 Nov 2009 23:17:01 +0000 (15:17 -0800)]
avoid infinite recursion in matcher.
after sync (or sync --local), clean up repository:
* look for and close CLs submitted on our behalf
* remove unmodified files from CLs
* warn about empty CLs
Robert Griesemer [Wed, 4 Nov 2009 21:31:20 +0000 (13:31 -0800)]
- always format blocks with closing } on a new line, even if empty
- at the moment this also affects single-line function declarations
because we have disabled them at the moment (but not single-line
function literals)
Russ Cox [Wed, 4 Nov 2009 11:15:24 +0000 (03:15 -0800)]
various tweaks to code review.
main one is to check at submit time that
user name being used in checkin message
is listed in the CONTRIBUTORS file.
this should catch misconfigurations.
another is to cut the @domain part
from the R= and CC= lines on checkin
messages, so that cc'ing someone on
a change does not mean their email
address is recorded for all time.
Adam Langley [Wed, 4 Nov 2009 01:23:50 +0000 (17:23 -0800)]
crypto/rsa: fix blinding when using a null random source.
For testing it can be useful to use a null random source (one which
always returns zero) to remove non-determinism from the tests.
However, when performing RSA blinding, the random blind ends up being
zero and it's hard to reverse a multiplication by zero.
Rob Pike [Wed, 4 Nov 2009 00:05:47 +0000 (16:05 -0800)]
First steps at command documentation: C compilers and linkers.
Each command gets a doc.go file for godoc to analyze. Its main
element is a package comment.
Robert Griesemer [Tue, 3 Nov 2009 21:40:11 +0000 (13:40 -0800)]
- don't loose extra line breaks in struct/interface declarations
- start new sections if a field/method declaration spans multiple lines;
this avoids tabs from the previous line affecting the next field/method
Robert Griesemer [Tue, 3 Nov 2009 18:00:57 +0000 (10:00 -0800)]
use default styler when there is no other given
(No visual change yet since the comments for
exported code snippets - as shown on the pkg
page - are not yet formatted using the standard
comment intersperse mechanism in the printer.
This is a TODO in the printer.)
Robert Griesemer [Tue, 3 Nov 2009 04:55:40 +0000 (20:55 -0800)]
split printer.go into two files; it has become too large:
- nodes.go implements ast node formatting
- printer.go implements the core printing functionality
and public interface
No code changes except for updating the import clauses
and adding a top-level comment to nodes.go.
Russ Cox [Tue, 3 Nov 2009 02:37:30 +0000 (18:37 -0800)]
package net cleanup
added ReadFrom/WriteTo for packet protocols like UDP.
simplified the net.Conn interface.
added new net.PacketConn interface for packet protocols.
implemented proper UDP listener.
cleaned up LocalAddr/RemoteAddr methods - cache in netFD.
threw away various unused methods.
an interface change:
introduced net.Addr as a network address interface,
to avoid conversion of UDP host:port to string and
back for every ReadFrom/WriteTo sequence.
another interface change:
since signature of Listener.Accept was changing anyway,
dropped the middle return value, because it is available
as c.RemoteAddr(). (the Accept signature predates the
existence of that method.)
Dial and Listen still accept strings, but the proto-specific
versions DialTCP, ListenUDP, etc. take net.Addr instead.
because the generic Dial didn't change and because
no one calls Accept directly (only indirectly via the http
server), very little code will be affected by these interface
changes.
Ian Lance Taylor [Mon, 2 Nov 2009 21:17:12 +0000 (13:17 -0800)]
Set LANG to control sorting. This affects the order in which
tests are processed, and thus the run.out output. The default
LANG on Fedora 10, en_US.utf8, causes the tests to be sorted
differently--e.g., arrayindex1.go and arrayindex.go are sorted
in the opposite order.