diff options
author | srinivasreddy <thatiparthysreenivas@gmail.com> | 2016-03-13 00:49:53 +0530 |
---|---|---|
committer | Steve Francia <steve.francia@gmail.com> | 2016-03-21 19:08:49 -0400 |
commit | 6258d48b0220fa59edf50a32031677eb7b7944d4 (patch) | |
tree | 23e549c375a5259442401e20756feb72b9c5e5c8 /utils | |
parent | c42982f76b397a5875114b59391712048e168348 (diff) |
Utils methods refactored for readability
Diffstat (limited to 'utils')
-rw-r--r-- | utils/utils.go | 50 |
1 files changed, 26 insertions, 24 deletions
diff --git a/utils/utils.go b/utils/utils.go index 3977a0e62..6f4264cca 100644 --- a/utils/utils.go +++ b/utils/utils.go @@ -20,35 +20,37 @@ import ( ) func CheckErr(err error, s ...string) { - if err != nil { - if len(s) == 0 { - jww.CRITICAL.Println(err) - } else { - for _, message := range s { - jww.ERROR.Println(message) - } - jww.ERROR.Println(err) - } + if err == nil { + return + } + if len(s) == 0 { + jww.CRITICAL.Println(err) + return } + for _, message := range s { + jww.ERROR.Println(message) + } + jww.ERROR.Println(err) } func StopOnErr(err error, s ...string) { - if err != nil { - if len(s) == 0 { - newMessage := err.Error() + if err == nil { + return + } - // Printing an empty string results in a error with - // no message, no bueno. - if newMessage != "" { - jww.CRITICAL.Println(newMessage) - } - } else { - for _, message := range s { - if message != "" { - jww.CRITICAL.Println(message) - } - } + defer os.Exit(-1) + + if len(s) == 0 { + newMessage := err.Error() + // Printing an empty string results in a error with + // no message, no bueno. + if newMessage != "" { + jww.CRITICAL.Println(newMessage) + } + } + for _, message := range s { + if message != "" { + jww.CRITICAL.Println(message) } - os.Exit(-1) } } |