Code review comment for lp:~niemeyer/juju-core/mstate-machine-watcher

Revision history for this message
William Reade (fwereade) wrote :

LGTM modulo locale worries.

https://codereview.appspot.com/6489104/diff/1/mstate/open.go
File mstate/open.go (right):

https://codereview.appspot.com/6489104/diff/1/mstate/open.go#newcode45
mstate/open.go:45: // Quite unfortunate that the error has no
appropriate code.
Very much so. Can we be screwed by locale settings here?

https://codereview.appspot.com/6489104/

« Back to merge proposal