summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorsrinivasreddy <thatiparthysreenivas@gmail.com>2016-03-13 00:49:53 +0530
committerSteve Francia <steve.francia@gmail.com>2016-03-21 19:08:49 -0400
commit6258d48b0220fa59edf50a32031677eb7b7944d4 (patch)
tree23e549c375a5259442401e20756feb72b9c5e5c8
parentc42982f76b397a5875114b59391712048e168348 (diff)
Utils methods refactored for readability
-rw-r--r--utils/utils.go50
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)
}
}