Remove dependence on MemberwiseClone. TODO: check if this is approach…

… is more performant
fsprojects · Oct 29, 2022 · e982267 · e982267
1 parent 43f311d
commit e982267
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/FSharpy.TaskSeq/TaskSeqBuilder.fs b/src/FSharpy.TaskSeq/TaskSeqBuilder.fs
@@ -263,8 +263,9 @@ and [<NoComparison; NoEquality>] TaskSeq<'Machine, 'T
                 //
                 // Solution: we shadow the initial machine, which we then re-assign here:
                 //
-                let clone = this.MemberwiseClone() :?> TaskSeq<'Machine, 'T>
-                clone._machine <- clone._initialMachine
+                let clone = TaskSeq<'Machine, 'T>() // we used MemberwiseClone, TODO: test difference in perf, but this should be faster
+                clone._machine <- this._initialMachine
+                clone._initialMachine <- this._initialMachine // TODO: proof with a test that this is necessary: probably not
                 clone.InitMachineData(ct, &clone._machine)
                 clone