Logging api

Refactorings to support testing logging interface. @bernerdschaefer @peterbourgon 
I understand that you like concision and that go is all about brevity, but
as someone who didn't write the code I found those names to be quite
impenetrable and a hindrance to comprehension.


On Tue, Aug 5, 2014 at 2:34 PM, Peter Bourgon <notifications@github.com>
wrote:

> In harpoon-agent/registry.go:
>
> >
> > -	go func(c *container, outc chan agent.ContainerInstance) {
> > -		inc := make(chan agent.ContainerInstance)
> > -		c.Subscribe(inc)
> > -		defer c.Unsubscribe(inc)
> > +	// Forward the container's state changes to all subscribers.
> > +	go func(c Container, outStatec chan agent.ContainerInstance) {
>
> Please revert to shorter names inc, outc
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/soundcloud/harpoon/pull/21/files#r15808776>.
>