madhatter22 / LinqToLdap

C# LINQ provider built on top of System.DirectoryServices.Protocols for querying and updating LDAP servers.
MIT License
45 stars 23 forks source link

Fix DelegateSpeedTest not running in all localisations #36

Open rklec opened 1 month ago

rklec commented 1 month ago

As , is a thousand separator (only) in (US) English or so, this test would fail in other localizations such as German with an exception while parsing.

AFAIK this is how you usually separate that in C# without having to resort to parsing static strings.

rklec commented 1 month ago

Oh, and please note this could (have skewed) the performance benchmarks, as it may have been parsed at each iteration (Actually not sure, but well at least once)? Now the looping is not parsed anymore.