Code review comment for lp:~waigani/juju-core/lxc-autorestart-setting

https://codereview.appspot.com/73390043/diff/1/container/lxc/lxc.go
File container/lxc/lxc.go (right):

https://codereview.appspot.com/73390043/diff/1/container/lxc/lxc.go#newcode70
container/lxc/lxc.go:70: }
Wouldn't this whole block be equivalent to:

autoRestart := conf["autoRestart"] == "" || conf["autoRestart"] ==
"true"

it seems a bit odd to me to do if true then true if false then false,
but it is just an opinion.

https://codereview.appspot.com/73390043/