GitLab

- distinguish log.Error and log.Errorf functions
- Initialize structs with field names
- A bit of unreachable code (defers)

dht doesn't need the whole network interface, only needs a Dialer.
(much reduced surface of possible errors)

The query-- once it's actually attempting to connect to a peer--
should be the one connecting.

Changed lots of peer use, and changed the peerstore to ensure
there is only ever one peer in use.

Fixed #174

![](http://m.memegen.com/77n7dk.jpg)

Now, all peers should be retrieved from the Peerstore, which will
construct the peers accordingly. This ensures there's only one peer
object per peer (opposite would be bad: things get out sync)

cc @whyrusleeping

@whyrusleeping pre-commit hook?

Otherwise don't have the peer's target address.

and other logging switches. I kept the u.PErr and u.POut in cli
commands, as those do need to write raw output directly.

- use log.* instead of u.*
- use automatic type conversions to .String()
  (Peer.String() prints nicely, and avoids calling b58 encoding
  until needed)