From 0d54677e138512cd5f403c6e48eaf51d9a511cf4 Mon Sep 17 00:00:00 2001
From: Baiju Muthukadan <baiju.m.mail@gmail.com>
Date: Fri, 7 Oct 2016 17:12:29 +0530
Subject: [PATCH] if no arguments, return error at once

---
 main.go | 60 ++++++++++++++++++++++++++++-----------------------------
 1 file changed, 30 insertions(+), 30 deletions(-)

diff --git a/main.go b/main.go
index 52a536d0..6f7e625e 100644
--- a/main.go
+++ b/main.go
@@ -44,40 +44,40 @@ func Open(dialect string, args ...interface{}) (*DB, error) {
 
 	if len(args) == 0 {
 		err = errors.New("invalid database source")
-	} else {
-		var source string
-		var dbSQL sqlCommon
+		return nil, err
+	}
+	var source string
+	var dbSQL sqlCommon
 
-		switch value := args[0].(type) {
-		case string:
-			var driver = dialect
-			if len(args) == 1 {
-				source = value
-			} else if len(args) >= 2 {
-				driver = value
-				source = args[1].(string)
-			}
-			dbSQL, err = sql.Open(driver, source)
-		case sqlCommon:
-			source = reflect.Indirect(reflect.ValueOf(value)).FieldByName("dsn").String()
-			dbSQL = value
+	switch value := args[0].(type) {
+	case string:
+		var driver = dialect
+		if len(args) == 1 {
+			source = value
+		} else if len(args) >= 2 {
+			driver = value
+			source = args[1].(string)
 		}
+		dbSQL, err = sql.Open(driver, source)
+	case sqlCommon:
+		source = reflect.Indirect(reflect.ValueOf(value)).FieldByName("dsn").String()
+		dbSQL = value
+	}
 
-		db = DB{
-			dialect:   newDialect(dialect, dbSQL.(*sql.DB)),
-			logger:    defaultLogger,
-			callbacks: DefaultCallback,
-			source:    source,
-			values:    map[string]interface{}{},
-			db:        dbSQL,
-		}
-		db.parent = &db
+	db = DB{
+		dialect:   newDialect(dialect, dbSQL.(*sql.DB)),
+		logger:    defaultLogger,
+		callbacks: DefaultCallback,
+		source:    source,
+		values:    map[string]interface{}{},
+		db:        dbSQL,
+	}
+	db.parent = &db
 
-		if err == nil {
-			err = db.DB().Ping() // Send a ping to make sure the database connection is alive.
-			if err != nil {
-				db.DB().Close()
-			}
+	if err == nil {
+		err = db.DB().Ping() // Send a ping to make sure the database connection is alive.
+		if err != nil {
+			db.DB().Close()
 		}
 	}