From 4ee222c5d57b34e0f5ea91acf5278d35420ee8d5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Eduardo=20C=C3=A1ceres?= Date: Fri, 8 Dec 2023 13:06:18 +0100 Subject: [PATCH] Remove Parallel.For, saving.. 80ms out of the 90 it was taking? Premature optimization is the root of blah, blah, blah :D --- src/AoC_2023/Day_08.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/AoC_2023/Day_08.cs b/src/AoC_2023/Day_08.cs index 6cce070..51f61e3 100644 --- a/src/AoC_2023/Day_08.cs +++ b/src/AoC_2023/Day_08.cs @@ -89,7 +89,7 @@ public ulong Solve_2_Original() var instruction = _input.Instructions[(int)(counter++ % (ulong)_input.Instructions.Length)]; finish = repetitionPeriodAfterEnd.All(p => p != default); - Parallel.For(0, currentNodeArray.Length, (i) => + for (int i = 0; i < currentNodeArray.Length; ++i) { currentNodeArray[i] = instruction switch { @@ -105,7 +105,7 @@ public ulong Solve_2_Original() repetitionPeriodAfterEnd[i] = counter - repetitionPeriodAfterEnd[i]; finish = false; } - }); + } } return SheepTools.Maths.LeastCommonMultiple(repetitionPeriodAfterEnd);