Fixed rcrowley#252 - guard against multiple registrations of runtime …

…and debug stats - gofmt code...

debug.go

@@ -2,8 +2,8 @@ package metrics
 
 import (
 	"runtime/debug"
-	"time"
 	"sync"
+	"time"
 )
 
 var (

debug_test.go

@@ -53,12 +53,12 @@ func TestDebugGCStatsDoubleRegister(t *testing.T) {
 	zero := debugMetrics.GCStats.NumGC.Value() // Get a "zero" since GC may have run before these tests.
 	runtime.GC()
 	CaptureDebugGCStatsOnce(r)
-	if numGC := debugMetrics.GCStats.NumGC.Value(); 1 != numGC - zero {
+	if numGC := debugMetrics.GCStats.NumGC.Value(); 1 != numGC-zero {
 		t.Errorf("NumGC got %d, expected 1", numGC)
 	}
 
 	RegisterDebugGCStats(r)
-	if numGC := debugMetrics.GCStats.NumGC.Value(); 1 != numGC - zero {
-		t.Errorf("NumGC got %d, expected 1", numGC - zero)
+	if numGC := debugMetrics.GCStats.NumGC.Value(); 1 != numGC-zero {
+		t.Errorf("NumGC got %d, expected 1", numGC-zero)
 	}
-}
+}

runtime.go

@@ -3,8 +3,8 @@ package metrics
 import (
 	"runtime"
 	"runtime/pprof"
-	"time"
 	"sync"
+	"time"
 )
 
 var (
@@ -50,7 +50,7 @@ var (
 	numGC       uint32
 	numCgoCalls int64
 
-	threadCreateProfile = pprof.Lookup("threadcreate")
+	threadCreateProfile        = pprof.Lookup("threadcreate")
 	registerRuntimeMetricsOnce = sync.Once{}
 )
 

runtime_test.go

@@ -94,12 +94,12 @@ func TestRuntimeMemStatsDoubleRegister(t *testing.T) {
 	runtime.GC()
 	CaptureRuntimeMemStatsOnce(r)
 
-	if count := runtimeMetrics.MemStats.NumGC.Value(); 1 != count - zero {
-		t.Errorf("NumGC got %d, expected 1", count - zero)
+	if count := runtimeMetrics.MemStats.NumGC.Value(); 1 != count-zero {
+		t.Errorf("NumGC got %d, expected 1", count-zero)
 	}
 
 	RegisterRuntimeMemStats(r)
-	if count := runtimeMetrics.MemStats.NumGC.Value(); 1 != count - zero {
-		t.Errorf("NumGC got %d, expected 1", count - zero)
+	if count := runtimeMetrics.MemStats.NumGC.Value(); 1 != count-zero {
+		t.Errorf("NumGC got %d, expected 1", count-zero)
 	}
-}
+}