[api-logs] Remove unneeded if check in LogRecordAttributeList (#6076)

Co-authored-by: Mikel Blanchard <[email protected]>
open-telemetry · Jan 22, 2025 · 48e5a08 · 48e5a08
1 parent fdca498
commit 48e5a08
Showing 1 changed file with 35 additions and 38 deletions.
diff --git a/src/OpenTelemetry.Api/Logs/LogRecordAttributeList.cs b/src/OpenTelemetry.Api/Logs/LogRecordAttributeList.cs
@@ -228,53 +228,50 @@ public readonly Enumerator GetEnumerator()
         attributeStorage ??= new List<KeyValuePair<string, object?>>(OverflowAdditionalCapacity);
 
         // TODO: Perf test this, adjust as needed.
-        if (count > 0)
+        attributeStorage.Add(this.attribute1);
+        if (count == 1)
         {
-            attributeStorage.Add(this.attribute1);
-            if (count == 1)
-            {
-                return attributeStorage;
-            }
-
-            attributeStorage.Add(this.attribute2);
-            if (count == 2)
-            {
-                return attributeStorage;
-            }
+            return attributeStorage;
+        }
 
-            attributeStorage.Add(this.attribute3);
-            if (count == 3)
-            {
-                return attributeStorage;
-            }
+        attributeStorage.Add(this.attribute2);
+        if (count == 2)
+        {
+            return attributeStorage;
+        }
 
-            attributeStorage.Add(this.attribute4);
-            if (count == 4)
-            {
-                return attributeStorage;
-            }
+        attributeStorage.Add(this.attribute3);
+        if (count == 3)
+        {
+            return attributeStorage;
+        }
 
-            attributeStorage.Add(this.attribute5);
-            if (count == 5)
-            {
-                return attributeStorage;
-            }
+        attributeStorage.Add(this.attribute4);
+        if (count == 4)
+        {
+            return attributeStorage;
+        }
 
-            attributeStorage.Add(this.attribute6);
-            if (count == 6)
-            {
-                return attributeStorage;
-            }
+        attributeStorage.Add(this.attribute5);
+        if (count == 5)
+        {
+            return attributeStorage;
+        }
 
-            attributeStorage.Add(this.attribute7);
-            if (count == 7)
-            {
-                return attributeStorage;
-            }
+        attributeStorage.Add(this.attribute6);
+        if (count == 6)
+        {
+            return attributeStorage;
+        }
 
-            attributeStorage.Add(this.attribute8);
+        attributeStorage.Add(this.attribute7);
+        if (count == 7)
+        {
+            return attributeStorage;
         }
 
+        attributeStorage.Add(this.attribute8);
+
         return attributeStorage;
     }