Update ArrayExtensions.swift (#1222)

fallwd · web-flow · commit 3ad58e4931f7 · 2024-11-20T15:37:15.000-03:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,8 @@ The changelog for **SwifterSwift**. Also see the [releases](https://github.com/S
 
 ### Fixed
 - **Examples** : Added "import Foundation" to Foundation Extension Playground page for Date() extension to work properly on Mac
+- **Array**
+  - `init(unsafeUninitializedCapacity:initializedWith:)` initializedCount should be eaqule to the number of successfully initialized elements, Ensure that the Array properly release allocated memory in case of an error.[#1222](https://github.com/SwifterSwift/SwifterSwift/pull/1222) by [fallwd](https://github.com/fallwd)
 
 ### Deprecated
 
diff --git a/Sources/SwifterSwift/SwiftStdlib/ArrayExtensions.swift b/Sources/SwifterSwift/SwiftStdlib/ArrayExtensions.swift
@@ -13,8 +13,8 @@ public extension Array {
         try self.init(unsafeUninitializedCapacity: count) { buffer, initializedCount in
             for index in 0..<count {
                 try buffer.baseAddress?.advanced(by: index).initialize(to: element(index))
+                initializedCount += 1
             }
-            initializedCount = count
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@ public extension Array {`
`13`	`13`	`try self.init(unsafeUninitializedCapacity: count) { buffer, initializedCount in`
`14`	`14`	`for index in 0..<count {`
`15`	`15`	`try buffer.baseAddress?.advanced(by: index).initialize(to: element(index))`
	`16`	`+ initializedCount += 1`
`16`	`17`	`}`
`17`		`- initializedCount = count`
`18`	`18`	`}`
`19`	`19`	`}`
`20`	`20`	`}`