[Monodevelop-patches-list] r2727 - trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding

[Lluis Sanchez <lluis@ximian.com>]

Author: lluis
Date: 2005-08-09 20:04:15 -0400 (Tue, 09 Aug 2005)
New Revision: 2727

Modified:
   trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/CSharpBindingCompilerManager.cs
   trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/ChangeLog
Log:
2005-08-06  Ben Motmans  <ben.motmans at gmail.com>

	* CSharpBindingCompilerManager.cs: correctly parse mcs errors.



Modified: trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/CSharpBindingCompilerManager.cs
===================================================================
--- trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/CSharpBindingCompilerManager.cs	2005-08-09 23:52:55 UTC (rev 2726)
+++ trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/CSharpBindingCompilerManager.cs	2005-08-10 00:04:15 UTC (rev 2727)
@@ -530,7 +530,7 @@
 		}
 
 		// Snatched from our codedom code :-).
-		static Regex regexError = new Regex (@"^(\s*(?<file>.*)\((?<line>\d*)(,(?<column>\d*))?\)\s+)*(?<level>\w+)\s*(?<number>.*):\s(?<message>.*)",
+		static Regex regexError = new Regex (@"^(\s*(?<file>.*)\((?<line>\d*)(,(?<column>\d*[\+]*))?\)(:|)\s+)*(?<level>\w+)\s*(?<number>.*):\s(?<message>.*)",
 			RegexOptions.Compiled | RegexOptions.ExplicitCapture);
 		
 		private static CompilerError CreateErrorFromString(string error_string)
@@ -550,8 +550,12 @@
 				error.FileName=match.Result("${file}");
 			if (String.Empty != match.Result("${line}"))
 				error.Line=Int32.Parse(match.Result("${line}"));
-			if (String.Empty != match.Result("${column}"))
-				error.Column=Int32.Parse(match.Result("${column}"));
+			if (String.Empty != match.Result("${column}")) {
+				if (match.Result("${column}") == "255+")
+					error.Column = -1;
+				else
+					error.Column=Int32.Parse(match.Result("${column}"));
+			}
 			if (match.Result("${level}")=="warning")
 				error.IsWarning=true;
 			error.ErrorNumber=match.Result("${number}");

Modified: trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/ChangeLog
===================================================================
--- trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/ChangeLog	2005-08-09 23:52:55 UTC (rev 2726)
+++ trunk/MonoDevelop/Core/src/AddIns/BackendBindings/CSharpBinding/ChangeLog	2005-08-10 00:04:15 UTC (rev 2727)
@@ -1,3 +1,7 @@
+2005-08-06  Ben Motmans  <ben.motmans at gmail.com>
+
+	* CSharpBindingCompilerManager.cs: correctly parse mcs errors.
+
 2005-08-09  Lluis Sanchez Gual  <lluis at novell.com> 
 
 	* Parser/Parser.cs: