state: give state backing its own file

state: give state backing its own file

The implementation is about to get
bigger, and it's conceptually separate
from the allWatcher itself.

https://code.launchpad.net/~rogpeppe/juju-core/270-megawatcher-backing-new-file/+merge/157403

(do not edit description out of merge proposal)

[rog, 2013-04-05 15:16:06]

Please take a look.

[gz, 2013-04-05 15:21:41]

LGTM

https://codereview.appspot.com/8422043/diff/2001/state/megabacking.go
File state/megabacking.go (right):

https://codereview.appspot.com/8422043/diff/2001/state/megabacking.go#newcode1
state/megabacking.go:1: package state
Would a top comment here roughly saying what the file is about make sense?

[rog, 2013-04-05 15:25:02]

On 2013/04/05 15:21:41, gz wrote:
> LGTM
> 
> https://codereview.appspot.com/8422043/diff/2001/state/megabacking.go
> File state/megabacking.go (right):
> 
> https://codereview.appspot.com/8422043/diff/2001/state/megabacking.go#newcode1
> state/megabacking.go:1: package state
> Would a top comment here roughly saying what the file is about make sense?

i'm not sure that i can say much in a file-level comment that the type decl and
doc comment at the top don't already say.

[rog, 2013-04-05 15:26:13]

*** Submitted:

state: give state backing its own file

The implementation is about to get
bigger, and it's conceptually separate
from the allWatcher itself.

R=gz
CC=
https://codereview.appspot.com/8422043