|
|
|
juju/api.go: actually handle error (bug #1312136)
(refactored from a change John made)
If there is a problem reading the .jenv file, it can return an error,
but we weren't paying attention to it. Instead we would end up getting a
panic() a few lines down when we go to use a *Config that is actually
nil.
addition by Nate:
Refactored the code to extract the logic for getting the config into a separate function, which cleans up the error handling significantly.
https://code.launchpad.net/~natefinch/juju-core/jameinel-1.18-panic-parsing-jenv-1312136/+merge/217084
(do not edit description out of merge proposal)
Patch Set 1 #Patch Set 2 : juju/api.go: actually handle error (bug #1312136) #
Total messages: 4
|
natefinch
Please take a look.
|
11 years, 8 months ago (2014年04月24日 16:17:27 UTC) #1 | ||||||||||||||||||||||||||||||||||||||||||||||||||||
Please take a look.
Please take a look.