clean code

petersunbag · Feb 11, 2018 · ac42a09 · ac42a09
1 parent bc55107
commit ac42a09
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/element.go b/element.go
@@ -54,15 +54,13 @@ func (e *elemConverter) convert(dPtr, sPtr unsafe.Pointer) {
 	}
 
 	deep := 0
-	oldPtr := dPtr
-	for ; deep < e.dReferDeep && dPtr != nil; deep++ {
-		oldPtr = dPtr
+	for ; deep < e.dReferDeep; deep++ {
+		oldPtr := dPtr
 		dPtr = unsafe.Pointer(*((**int)(dPtr)))
-	}
-	// if dPtr is nil, for loop above must have executed at least once
-	if dPtr == nil {
-		dPtr = oldPtr
-		deep--
+		if dPtr == nil {
+			dPtr = oldPtr
+			break
+		}
 	}
 
 	if deep := e.dReferDeep - deep; deep > 0 {