shithub: hugo

Download patch

ref: 6258d48b0220fa59edf50a32031677eb7b7944d4
parent: c42982f76b397a5875114b59391712048e168348
author: srinivasreddy <[email protected]>
date: Sat Mar 12 19:49:53 EST 2016

Utils methods refactored for readability

--- a/utils/utils.go
+++ b/utils/utils.go
@@ -20,35 +20,37 @@
 )
 
 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)
 		}
-		os.Exit(-1)
+	}
+	for _, message := range s {
+		if message != "" {
+			jww.CRITICAL.Println(message)
+		}
 	}
 }