From f145e1042fd220664f13c564f439723312a07311 Mon Sep 17 00:00:00 2001
From: Harshavardhana <harsha@minio.io>
Date: Wed, 4 May 2016 20:22:15 -0700
Subject: [PATCH] quick/config: No need to use Data() with type assertion.
 (#1480)

Since input to quick.New() is a pointer the unmarshalled value
internally already has the value, subsequent type assertions
are not needed.

Thanks to Bala for finding this behavior.

Fixes #1475
---
 config-old.go | 24 ++++++++++++------------
 config-v4.go  |  2 +-
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/config-old.go b/config-old.go
index be618e98c..eaa0d29e5 100644
--- a/config-old.go
+++ b/config-old.go
@@ -24,16 +24,16 @@ func loadConfigV1() (*configV1, error) {
 	if _, err = os.Stat(configFile); err != nil {
 		return nil, err
 	}
-	a := &configV1{}
-	a.Version = "1"
-	qc, err := quick.New(a)
+	c := &configV1{}
+	c.Version = "1"
+	qc, err := quick.New(c)
 	if err != nil {
 		return nil, err
 	}
 	if err := qc.Load(configFile); err != nil {
 		return nil, err
 	}
-	return qc.Data().(*configV1), nil
+	return c, nil
 }
 
 /////////////////// Config V2 ///////////////////
@@ -67,16 +67,16 @@ func loadConfigV2() (*configV2, error) {
 	if _, err = os.Stat(configFile); err != nil {
 		return nil, err
 	}
-	a := &configV2{}
-	a.Version = "2"
-	qc, err := quick.New(a)
+	c := &configV2{}
+	c.Version = "2"
+	qc, err := quick.New(c)
 	if err != nil {
 		return nil, err
 	}
 	if err := qc.Load(configFile); err != nil {
 		return nil, err
 	}
-	return qc.Data().(*configV2), nil
+	return c, nil
 }
 
 /////////////////// Config V3 ///////////////////
@@ -134,14 +134,14 @@ func loadConfigV3() (*configV3, error) {
 	if _, err = os.Stat(configFile); err != nil {
 		return nil, err
 	}
-	a := &configV3{}
-	a.Version = "3"
-	qc, err := quick.New(a)
+	c := &configV3{}
+	c.Version = "3"
+	qc, err := quick.New(c)
 	if err != nil {
 		return nil, err
 	}
 	if err := qc.Load(configFile); err != nil {
 		return nil, err
 	}
-	return qc.Data().(*configV3), nil
+	return c, nil
 }
diff --git a/config-v4.go b/config-v4.go
index 1985a1227..abe4167a6 100644
--- a/config-v4.go
+++ b/config-v4.go
@@ -87,7 +87,7 @@ func initConfig() error {
 		return err
 	}
 	// Save the loaded config globally.
-	serverConfig = qc.Data().(*serverConfigV4)
+	serverConfig = srvCfg
 	// Set the version properly after the unmarshalled json is loaded.
 	serverConfig.Version = globalMinioConfigVersion
 	return nil