Merge pull request #21 from do0om/develop

siywilliams · siywilliams · commit 4bcd6a2f60fc · 2013-06-28T16:39:39.000-07:00
Fixed double parsing on french computers where "125.23" wasn't parsed as...
diff --git a/FluentCommandLineParser/Internals/Parsing/OptionParsers/DoubleCommandLineOptionParser.cs b/FluentCommandLineParser/Internals/Parsing/OptionParsers/DoubleCommandLineOptionParser.cs
@@ -38,7 +38,7 @@ public class DoubleCommandLineOptionParser : ICommandLineOptionParser<double>
 		/// <returns></returns>
 		public double Parse(ParsedOption parsedOption)
 		{
-			return double.Parse(parsedOption.Value, CultureInfo.CurrentCulture);
+			return double.Parse(parsedOption.Value, CultureInfo.InvariantCulture);
 		}
 
 		/// <summary>
@@ -49,7 +49,7 @@ public double Parse(ParsedOption parsedOption)
 		public bool CanParse(ParsedOption parsedOption)
 		{
 			double result;
-			return double.TryParse(parsedOption.Value, out result);
+            return double.TryParse(parsedOption.Value, System.Globalization.NumberStyles.Number, CultureInfo.InvariantCulture, out result);
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public class DoubleCommandLineOptionParser : ICommandLineOptionParser<double>`
`38`	`38`	`/// <returns></returns>`
`39`	`39`	`public double Parse(ParsedOption parsedOption)`
`40`	`40`	`{`
`41`		`- return double.Parse(parsedOption.Value, CultureInfo.CurrentCulture);`
	`41`	`+ return double.Parse(parsedOption.Value, CultureInfo.InvariantCulture);`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`/// <summary>`
`@@ -49,7 +49,7 @@ public double Parse(ParsedOption parsedOption)`
`49`	`49`	`public bool CanParse(ParsedOption parsedOption)`
`50`	`50`	`{`
`51`	`51`	`double result;`
`52`		`- return double.TryParse(parsedOption.Value, out result);`
	`52`	`+ return double.TryParse(parsedOption.Value, System.Globalization.NumberStyles.Number, CultureInfo.InvariantCulture, out result);`
`53`	`53`	`}`
`54`	`54`	`}`
`55`	`55`	`}`