Fixed whitespace, commented out dead code

Author: KathleenDollard

src/System.CommandLine/Parsing/CommandResult.cs

@@ -67,38 +67,39 @@ public CommandValueResult CommandValueResult
         /// <inheritdoc/>
         public override string ToString() => $"{nameof(CommandResult)}: {IdentifierToken.Value} {string.Join(" ", Tokens.Select(t => t.Value))}";
 
+        // TODO: DefaultValues
+        /*
         internal override bool UseDefaultValueFor(ArgumentResult argumentResult)
             => argumentResult.Argument.HasDefaultValue && argumentResult.Tokens.Count == 0;
+        */
 
+        // TODO: Validation
+        /*
         /// <param name="completeValidation">Only the inner most command goes through complete validation.</param>
         internal void Validate(bool completeValidation)
         {
             if (completeValidation)
             {
-                // TODO: invocation
-                //                if (Command.Action is null && Command.HasSubcommands)
                 if (Command.HasSubcommands)
                 {
                     SymbolResultTree.InsertFirstError(
                         new ParseError(LocalizationResources.RequiredCommandWasNotProvided(), this));
                 }
 
                 // TODO: validators
-                /*
-                                if (Command.HasValidators)
-                                {
-                                    int errorCountBefore = SymbolResultTree.ErrorCount;
-                                    for (var i = 0; i < Command.Validators.Count; i++)
-                                    {
-                                        Command.Validators[i](this);
-                                    }
-
-                                    if (SymbolResultTree.ErrorCount != errorCountBefore)
-                                    {
-                                        return;
-                                    }
-                                }
-                */
+                if (Command.HasValidators)
+                {
+                    int errorCountBefore = SymbolResultTree.ErrorCount;
+                    for (var i = 0; i < Command.Validators.Count; i++)
+                    {
+                        Command.Validators[i](this);
+                    }
+
+                    if (SymbolResultTree.ErrorCount != errorCountBefore)
+                    {
+                        return;
+                    }
+                }
             }
 
             // TODO: Validation
@@ -121,7 +122,7 @@ private void ValidateOptions(bool completeValidation)
                 var option = options[i];
 
                 // TODO: VersionOption, recursive options
-                //              if (!completeValidation && !(option.Recursive || option.Argument.HasDefaultValue || option is VersionOption))
+                // if (!completeValidation && !(option.Recursive || option.Argument.HasDefaultValue || option is VersionOption))
                 if (!completeValidation && !option.Argument.HasDefaultValue)
                 {
                     continue;
@@ -165,27 +166,23 @@ private void ValidateOptions(bool completeValidation)
                 }
 
                 // TODO: validators
-                /*
-                                if (optionResult.Option.HasValidators)
-                                {
-                                    int errorsBefore = SymbolResultTree.ErrorCount;
-
-                                    for (var j = 0; j < optionResult.Option.Validators.Count; j++)
-                                    {
-                                        optionResult.Option.Validators[j](optionResult);
-                                    }
-
-                                    if (errorsBefore != SymbolResultTree.ErrorCount)
-                                    {
-                                        continue;
-                                    }
-                                }
-                */
+                if (optionResult.Option.HasValidators)
+                {
+                    int errorsBefore = SymbolResultTree.ErrorCount;
+
+                    for (var j = 0; j < optionResult.Option.Validators.Count; j++)
+                    {
+                        optionResult.Option.Validators[j](optionResult);
+                    }
+
+                    if (errorsBefore != SymbolResultTree.ErrorCount)
+                    {
+                        continue;
+                    }
+                }
 
                 // TODO: Ensure all argument conversions are run for entered values
-                /*
                 _ = argumentResult.GetArgumentConversionResult();
-                */
             }
         }
 
@@ -226,5 +223,6 @@ private void ValidateArguments(bool completeValidation)
                 _ = argumentResult.GetArgumentConversionResult();
             }
         }
+        */
     }
 }

src/System.CommandLine/Parsing/OptionResult.cs

@@ -93,6 +93,9 @@ internal bool IsArgumentLimitReached
         internal ArgumentConversionResult ArgumentConversionResult
             => _argumentConversionResult ??= GetResult(Option.Argument)!.GetArgumentConversionResult();
 
+        // TODO: Default values
+        /*
         internal override bool UseDefaultValueFor(ArgumentResult argument) => Implicit;
+        */
     }
 }

src/System.CommandLine/Parsing/ParseOperation.cs

@@ -389,6 +389,8 @@ private void AddCurrentTokenToUnmatched()
             _symbolResultTree.AddUnmatchedToken(CurrentToken, _innermostCommandResult, _rootCommandResult);
         }
 
+        // TODO: Validation
+        /*
         private void Validate()
         {
             // Only the inner most command goes through complete validation,
@@ -403,5 +405,6 @@ private void Validate()
                 currentResult = currentResult.Parent as CommandResult;
             }
         }
+        */
     }
 }

src/System.CommandLine/Parsing/SymbolResult.cs

@@ -156,6 +156,9 @@ public IEnumerable<ParseError> Errors
         }
         */
 
+        // TODO: DefaultValues
+        /*
         internal virtual bool UseDefaultValueFor(ArgumentResult argumentResult) => false;
+        */
     }
 }