Fixd bug where some shortened filenames would be reserved names such as "con".

AArnott · AArnott · commit 82e155ebfbd6 · 2010-02-20T11:59:58.000-08:00
diff --git a/src/DotNetOpenAuth.BuildTasks/PathSegment.cs b/src/DotNetOpenAuth.BuildTasks/PathSegment.cs
@@ -20,6 +20,7 @@ internal class PathSegment {
 		private readonly string originalName;
 		private string currentName;
 		private bool minimized;
+		private static readonly string[] ReservedFileNames = "CON PRN AUX CLOCK$ NUL COM0 COM1 COM2 COM3 COM4 COM5 COM6 COM7 COM8 COM9 LPT0 LPT1 LPT2 LPT3 LPT4 LPT5 LPT6 LPT7 LPT8 LPT9".Split(' ');
 
 		internal PathSegment() {
 			this.currentName = string.Empty;
@@ -232,7 +233,7 @@ private string GetUniqueShortName(string preferredPrefix, string preferredSuffix
 			Contract.Ensures(Contract.Result<string>().Length <= allowableLength);
 			string candidateName = string.Empty;
 			int i;
-			for (i = -1; candidateName.Length == 0 || this.Siblings.Any(child => string.Equals(child.CurrentName, candidateName, StringComparison.OrdinalIgnoreCase)); i++) {
+			for (i = -1; candidateName.Length == 0 || ReservedFileNames.Contains(candidateName, StringComparer.OrdinalIgnoreCase) || this.Siblings.Any(child => string.Equals(child.CurrentName, candidateName, StringComparison.OrdinalIgnoreCase)); i++) {
 				string unique = i < 0 ? string.Empty : i.ToString("x");
 				if (allowableLength < unique.Length) {
 					throw new InvalidOperationException("Unable to shorten path sufficiently to fit constraints.");
