Fix concurrent list

jtheisen · Jan 25, 2023 · ef0dd78 · ef0dd78
1 parent 195a17c
commit ef0dd78
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/Moldinium/Misc/ConcurrentList.cs b/Moldinium/Misc/ConcurrentList.cs
@@ -4,7 +4,7 @@
 
 namespace Moldinium.Misc;
 
-struct LockHelper : IDisposable
+class LockHelper : IDisposable
 {
     SpinLock spinLock;
 
@@ -36,11 +36,6 @@ public void Dispose()
     }
 }
 
-ref struct LockHelperRef
-{
-    public ref int lockHelper;
-}
-
 /// <summary>
 /// A simple but likely quite inefficient implementation of a concurrent list that
 /// is meant only for demonstration purposes.
@@ -49,7 +44,7 @@ public class ConcurrentList<T> : IList<T>
 {
     IImmutableList<T> list = ImmutableList<T>.Empty;
 
-    LockHelper locker;
+    LockHelper locker = new LockHelper();
 
 	public T this[int index]
     {
@@ -70,7 +65,7 @@ public void Add(T item)
     {
         using var _ = locker.Lock();
 
-        list.Add(item);
+        list = list.Add(item);
     }
 
     public void Clear()

diff --git a/SampleApp.AspNetMvc/SampleApp.AspNetMvc.csproj b/SampleApp.AspNetMvc/SampleApp.AspNetMvc.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Web">
 
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
+    <TargetFramework>net7.0</TargetFramework>
     <Nullable>enable</Nullable>
     <ImplicitUsings>enable</ImplicitUsings>
   </PropertyGroup>