This was noted today when the patch was sent with the following line:
lblCompletionTime.Text = String.Concat(trainingSkill.EndTime.ToLocalTime())
I can understand why the contributor used this syntax, like the line above, concatenated two lines to form a multi-point date / time string.
Is there any hidden reason for a single overload of the String.Concat () parameter or was it included for "completeness" by the language developers.
I replaced the line:
lblCompletionTime.Text = trainingSkill.EndTime.ToLocalTime().ToString(CultureInfo.CurrentCulture)
which has the same result.
source
share