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
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/