Fixed NRT error

KathleenDollard · KathleenDollard · commit e1bf3f5d0897 · 2024-08-16T17:05:40.000-04:00
diff --git a/src/System.CommandLine/Parsing/CliCommandResultInternal.cs b/src/System.CommandLine/Parsing/CliCommandResultInternal.cs
@@ -47,10 +47,9 @@ public CliCommandResult CommandResult
                 if (commandResult is null)
                 {
                     var parent = Parent is CliCommandResultInternal commandResultInternal
-                        ? commandResultInternal.CommandResult 
+                        ? commandResultInternal.CommandResult
                         : null;
-                    // TODO: Resolve the NRT warning for locations
-                    commandResult = new CliCommandResult(Command, parent?.Locations, parent);
+                    commandResult = new CliCommandResult(Command, Tokens.Select(t => t.Location), parent);
                 }
                 // Reset unless we put tests in place to ensure it is not called in error handling before SymbolTree processing is complete
                 commandResult.ValueResults = Children.Select(GetValueResult).OfType<CliValueResult>().ToList();

Original file line number	Diff line number	Diff line change
`@@ -47,10 +47,9 @@ public CliCommandResult CommandResult`
`47`	`47`	`if (commandResult is null)`
`48`	`48`	`{`
`49`	`49`	`var parent = Parent is CliCommandResultInternal commandResultInternal`
`50`		`- ? commandResultInternal.CommandResult`
	`50`	`+ ? commandResultInternal.CommandResult`
`51`	`51`	`: null;`
`52`		`- // TODO: Resolve the NRT warning for locations`
`53`		`- commandResult = new CliCommandResult(Command, parent?.Locations, parent);`
	`52`	`+ commandResult = new CliCommandResult(Command, Tokens.Select(t => t.Location), parent);`
`54`	`53`	`}`
`55`	`54`	`// Reset unless we put tests in place to ensure it is not called in error handling before SymbolTree processing is complete`
`56`	`55`	`commandResult.ValueResults = Children.Select(GetValueResult).OfType<CliValueResult>().ToList();`