Fix CommandArgumentEnumerator behavior for .NET 8

natemcmaster · Dec 31, 2023 · e44d5fc · e44d5fc
1 parent 14f9a49
commit e44d5fc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/CommandLineUtils/Internal/CommandLineProcessor.cs b/src/CommandLineUtils/Internal/CommandLineProcessor.cs
@@ -525,7 +525,7 @@ public CommandArgumentEnumerator(IEnumerator<CommandArgument> enumerator)
                 _enumerator = enumerator;
             }
 
-            public CommandArgument Current => _enumerator.Current;
+            public CommandArgument Current { get; private set; }
 
             object IEnumerator.Current => Current;
 
@@ -535,7 +535,10 @@ public bool MoveNext()
             {
                 if (Current == null || !Current.MultipleValues)
                 {
-                    return _enumerator.MoveNext();
+                    var result = _enumerator.MoveNext();
+                    if (result)
+                        Current = _enumerator.Current!;
+                    return result;
                 }
 
                 // If current argument allows multiple values, we don't move forward and

diff --git a/test/CommandLineUtils.Tests/McMaster.Extensions.CommandLineUtils.Tests.csproj b/test/CommandLineUtils.Tests/McMaster.Extensions.CommandLineUtils.Tests.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net7.0;net6.0</TargetFrameworks>
+    <TargetFrameworks>net8.0;net7.0;net6.0</TargetFrameworks>
     <TargetFrameworks Condition="'$(TestFullFramework)' != 'false'">$(TargetFrameworks);net472</TargetFrameworks>
     <!-- Once xunit supports nullable reference types, I might reconsider -->
     <Nullable>annotations</Nullable>