X-Git-Url: http://wien.tomnetworks.com/gitweb/?a=blobdiff_plain;f=mcs%2Fmcs%2Freport.cs;h=3ca7fd681ea6c8f72c30415f38fc3327156da84d;hb=732841a4f171062563a831fb8cd3165be1ddfd6e;hp=b9008fcceb8a2d2df71555f03df518f1ed30bb00;hpb=7755c3d7d726d5d85a79763a15f7cc998956ac14;p=mono.git

diff --git a/mcs/mcs/report.cs b/mcs/mcs/report.cs
index b9008fcceb8..3ca7fd681ea 100644
--- a/mcs/mcs/report.cs
+++ b/mcs/mcs/report.cs
@@ -10,79 +10,44 @@
 using System;
 using System.IO;
 using System.Text;
-using System.Collections;
-using System.Collections.Specialized;
+using System.Collections.Generic;
 using System.Diagnostics;
 using System.Reflection;
 using System.Reflection.Emit;
 
 namespace Mono.CSharp {
 
-	/// <summary>
-	///   This class is used to report errors and warnings t te user.
-	/// </summary>
-	public class Report {
-		/// <summary>  
-		///   Errors encountered so far
-		/// </summary>
-		static public int Errors;
-
-		/// <summary>  
-		///   Warnings encountered so far
-		/// </summary>
-		static public int Warnings;
-
-		/// <summary>  
-		///   Whether errors should be throw an exception
-		/// </summary>
-		static public bool Fatal;
-		
+	//
+	// Errors and warnings manager
+	//
+	public class Report
+	{
 		/// <summary>  
 		///   Whether warnings should be considered errors
 		/// </summary>
-		static public bool WarningsAreErrors;
-		static ArrayList warnings_as_error;
-		static ArrayList warnings_only;
-
-		/// <summary>  
-		///   Whether to dump a stack trace on errors. 
-		/// </summary>
-		static public bool Stacktrace;
+		public bool WarningsAreErrors;
+		List<int> warnings_as_error;
+		List<int> warnings_only;
 
-		static public TextWriter Stderr = Console.Error;
-		
-		//
-		// If the 'expected' error code is reported then the
-                // compilation succeeds.
-		//
-		// Used for the test suite to excercise the error codes
-		//
-		static int expected_error = 0;
+		public static int DebugFlags = 0;
 
 		//
 		// Keeps track of the warnings that we are ignoring
 		//
-		public static Hashtable warning_ignore_table;
+		public Dictionary<int, bool> warning_ignore_table;
 
-		static Hashtable warning_regions_table;
+		Dictionary<string, WarningRegions> warning_regions_table;
 
-		//
-		// This is used to save/restore the error state.  When the
-		// error stack contains elements, warnings and errors are not
-		// reported to the user.  This is used by the Lambda expression
-		// support to compile the code with various parameter values.
-		// A stack because of `Report.Errors == errors;'
-		//
-		static Stack error_stack;
-		static Stack warning_stack;
-		static bool reporting_disabled;
-		
-		static int warning_level;
+		int warning_level;
+
+		ReportPrinter printer;
+
+		int reporting_disabled;
 		
 		/// <summary>
 		/// List of symbols related to reported error/warning. You have to fill it before error/warning is reported.
 		/// </summary>
-		static ArrayList extra_information = new ArrayList ();
+		List<string> extra_information = new List<string> ();
 
 		// 
 		// IF YOU ADD A NEW WARNING YOU HAVE TO ADD ITS ID HERE
@@ -91,23 +56,18 @@ namespace Mono.CSharp {
 			28, 67, 78,
 			105, 108, 109, 114, 162, 164, 168, 169, 183, 184, 197,
 			219, 251, 252, 253, 278, 282,
-			419, 420, 429, 436, 440, 465, 467, 469, 472,
-			612, 618, 626, 628, 642, 649, 652, 658, 659, 660, 661, 665, 672, 675,
+			402, 414, 419, 420, 429, 436, 440, 458, 464, 465, 467, 469, 472,
+			612, 618, 626, 628, 642, 649, 652, 658, 659, 660, 661, 665, 672, 675, 693,
 			809,
-			1030, 1066,
+			1030, 1058, 1066,
 			1522, 1570, 1571, 1572, 1573, 1574, 1580, 1581, 1584, 1587, 1589, 1590, 1591, 1592,
 			1616, 1633, 1634, 1635, 1685, 1690, 1691, 1692,
-			1717, 1718, 1720,
+			1700, 1717, 1718, 1720,
 			1901,
 			2002, 2023, 2029,
 			3000, 3001, 3002, 3003, 3005, 3006, 3007, 3008, 3009,
 			3010, 3011, 3012, 3013, 3014, 3015, 3016, 3017, 3018, 3019,
-			3021, 3022, 3023, 3026, 3027,
-			
-			414,	// Non ISO-1 warnings
-#if GMCS_SOURCE
-			402, 458, 464, 693, 1058, 1700, 3024
-#endif
+			3021, 3022, 3023, 3024, 3026, 3027
 		};
 
 		static Report ()
@@ -116,872 +76,872 @@ namespace Mono.CSharp {
 			Array.Sort (AllWarnings);
 		}
 
-		public static void Reset ()
+		public Report (ReportPrinter printer)
 		{
-			Errors = Warnings = 0;
-			WarningsAreErrors = false;
-			warning_ignore_table = null;
-			warning_regions_table = null;
-			reporting_disabled = false;
-			error_stack = warning_stack = null;
-			warnings_as_error = null;
-			warnings_only = null;
+			if (printer == null)
+				throw new ArgumentNullException ("printer");
+
+			this.printer = printer;
+			warning_level = 4;
 		}
 
-		public static void DisableReporting ()
+		public void DisableReporting ()
 		{
-			if (error_stack == null)
-				error_stack = new Stack ();
-			error_stack.Push (Errors);
-
-			if (Warnings > 0) {
-				if (warning_stack == null)
-					warning_stack = new Stack ();
-				warning_stack.Push (Warnings);
-			}
-
-			reporting_disabled = true;
+			++reporting_disabled;
 		}
 
-		public static void EnableReporting ()
+		public void EnableReporting ()
 		{
-			if (warning_stack != null && warning_stack.Count > 0)
-				Warnings = (int) warning_stack.Pop ();
-			else
-				Warnings = 0;
-
-			Errors = (int) error_stack.Pop ();
-			if (error_stack.Count == 0) {
-				reporting_disabled = false;
-			}
+			--reporting_disabled;
 		}
 
-		public static IMessageRecorder msg_recorder;
-
-		public static IMessageRecorder SetMessageRecorder (IMessageRecorder recorder)
+		public void FeatureIsNotAvailable (Location loc, string feature)
 		{
-			IMessageRecorder previous = msg_recorder;
-			msg_recorder = recorder;
-			return previous;
+			string version;
+			switch (RootContext.Version) {
+			case LanguageVersion.ISO_1:
+				version = "1.0";
+				break;
+			case LanguageVersion.ISO_2:
+				version = "2.0";
+				break;
+			case LanguageVersion.V_3:
+				version = "3.0";
+				break;
+			default:
+				throw new InternalErrorException ("Invalid feature version", RootContext.Version);
+			}
+
+			Error (1644, loc,
+				"Feature `{0}' cannot be used because it is not part of the C# {1} language specification",
+				      feature, version);
 		}
 
-		public interface IMessageRecorder
+		public void FeatureIsNotSupported (Location loc, string feature)
 		{
-			bool IsEmpty { get; }
-			void EndSession ();
-			void AddMessage (AbstractMessage msg);
-			bool PrintMessages ();
+			Error (1644, loc,
+				"Feature `{0}' is not supported in Mono mcs1 compiler. Consider using the `gmcs' compiler instead",
+				feature);
+		}
+		
+		static bool IsValidWarning (int code)
+		{	
+			return Array.BinarySearch (AllWarnings, code) >= 0;
 		}
 
-		//
-		// Default message recorder, it uses two types of message groups.
-		// Common messages: messages reported in all sessions.
-		// Merged messages: union of all messages in all sessions. 
-		//		
-		public struct MessageRecorder : IMessageRecorder
+		bool IsWarningEnabled (int code, int level, Location loc)
 		{
-			ArrayList session_messages;
-			//
-			// A collection of exactly same messages reported in all sessions
-			//
-			ArrayList common_messages;
-
-			//
-			// A collection of unique messages reported in all sessions
-			//
-			ArrayList merged_messages;
-
-			public void EndSession ()
-			{
-				if (session_messages == null)
-					return;
-
-				//
-				// Handles the first session
-				//
-				if (common_messages == null) {
-					common_messages = new ArrayList (session_messages);
-					merged_messages = session_messages;
-					session_messages = null;
-					return;
-				}
-
-				//
-				// Store common messages if any
-				//
-				for (int i = 0; i < common_messages.Count; ++i) {
-					AbstractMessage cmsg = (AbstractMessage) common_messages [i];
-					bool common_msg_found = false;
-					foreach (AbstractMessage msg in session_messages) {
-						if (cmsg.Equals (msg)) {
-							common_msg_found = true;
-							break;
-						}
-					}
-
-					if (!common_msg_found)
-						common_messages.RemoveAt (i);
-				}
-
-				//
-				// Merge session and previous messages
-				//
-				for (int i = 0; i < session_messages.Count; ++i) {
-					AbstractMessage msg = (AbstractMessage) session_messages [i];
-					bool msg_found = false;
-					for (int ii = 0; ii < merged_messages.Count; ++ii) {
-						if (msg.Equals (merged_messages [ii])) {
-							msg_found = true;
-							break;
-						}
-					}
+			if (WarningLevel < level)
+				return false;
 
-					if (!msg_found)
-						merged_messages.Add (msg);
+			if (warning_ignore_table != null) {
+				if (warning_ignore_table.ContainsKey (code)) {
+					return false;
 				}
 			}
 
-			public void AddMessage (AbstractMessage msg)
-			{
-				if (session_messages == null)
-					session_messages = new ArrayList ();
+			if (warning_regions_table == null || loc.IsNull)
+				return true;
 
-				session_messages.Add (msg);
-			}
+			WarningRegions regions;
+			if (!warning_regions_table.TryGetValue (loc.Name, out regions))
+				return true;
 
-			public bool IsEmpty {
-				get {
-					return merged_messages == null && common_messages == null;
-				}
-			}
+			return regions.IsWarningEnabled (code, loc.Row);
+		}
 
-			//
-			// Prints collected messages, common messages have a priority
-			//
-			public bool PrintMessages ()
-			{
-				ArrayList messages_to_print = merged_messages;
-				if (common_messages != null && common_messages.Count > 0) {
-					messages_to_print = common_messages;
-				}
+		bool IsWarningAsError (int code)
+		{
+			bool is_error = WarningsAreErrors;
 
-				if (messages_to_print == null)
-					return false;
+			// Check specific list
+			if (warnings_as_error != null)
+				is_error |= warnings_as_error.Contains (code);
 
-				foreach (AbstractMessage msg in messages_to_print)
-					msg.Print ();
+			// Ignore excluded warnings
+			if (warnings_only != null && warnings_only.Contains (code))
+				is_error = false;
 
-				return true;
-			}
+			return is_error;
 		}
-		
-		public abstract class AbstractMessage
+		        
+		public void RuntimeMissingSupport (Location loc, string feature) 
 		{
-			readonly string[] extra_info;
-			protected readonly int code;
-			protected readonly Location location;
-			readonly string message;
+			Error (-88, loc, "Your .NET Runtime does not support `{0}'. Please use the latest Mono runtime instead.", feature);
+		}
 
-			protected AbstractMessage (int code, Location loc, string msg, ArrayList extraInfo)
-			{
-				this.code = code;
-				if (code < 0)
-					this.code = 8000 - code;
+		/// <summary>
+		/// In most error cases is very useful to have information about symbol that caused the error.
+		/// Call this method before you call Report.Error when it makes sense.
+		/// </summary>
+		public void SymbolRelatedToPreviousError (Location loc, string symbol)
+		{
+			SymbolRelatedToPreviousError (loc.ToString (), symbol);
+		}
 
-				this.location = loc;
-				this.message = msg;
-				if (extraInfo.Count != 0) {
-					this.extra_info = (string[])extraInfo.ToArray (typeof (string));
-				}
-			}
+		public void SymbolRelatedToPreviousError (MemberSpec ms)
+		{
+			if (reporting_disabled > 0 || !printer.HasRelatedSymbolSupport)
+				return;
 
-			protected AbstractMessage (AbstractMessage aMsg)
-			{
-				this.code = aMsg.code;
-				this.location = aMsg.location;
-				this.message = aMsg.message;
-				this.extra_info = aMsg.extra_info;
+			var mc = ms.MemberDefinition as MemberCore;
+			if (mc != null) {
+				SymbolRelatedToPreviousError (mc);
+			} else {
+				var im = ms.MemberDefinition as ImportedMemberDefinition;
+				SymbolRelatedToPreviousError (im.Assembly.Location, "");
 			}
+		}
 
-			static void Check (int code)
-			{
-				if (code == expected_error) {
-					Environment.Exit (0);
-				}
-			}
+		public void SymbolRelatedToPreviousError (MemberCore mc)
+		{
+			SymbolRelatedToPreviousError (mc.Location, mc.GetSignatureForError ());
+		}
 
-			public override bool Equals (object obj)
-			{
-				AbstractMessage msg = obj as AbstractMessage;
-				if (msg == null)
-					return false;
+		void SymbolRelatedToPreviousError (string loc, string symbol)
+		{
+			string msg = String.Format ("{0} (Location of the symbol related to previous ", loc);
+			if (extra_information.Contains (msg))
+				return;
 
-				return code == msg.code && location.Equals (msg.location) && message == msg.message;
-			}
+			extra_information.Add (msg);
+		}
 
-			public override int GetHashCode ()
-			{
-				return code.GetHashCode ();
+		public void AddWarningAsError (string warningId)
+		{
+			int id;
+			try {
+				id = int.Parse (warningId);
+			} catch {
+				id = -1;
 			}
 
-			public abstract bool IsWarning { get; }
-
-			public abstract string MessageType { get; }
-
-			public virtual void Print ()
-			{
-				if (msg_recorder != null) {
-					//
-					// This line is useful when debugging messages recorder
-					//
-					// Console.WriteLine ("RECORDING: {0} {1} {2}", code, location, message);
-					msg_recorder.AddMessage (this);
-					return;
-				}
+			if (!CheckWarningCode (id, warningId, Location.Null))
+				return;
 
-				if (reporting_disabled)
-					return;
+			if (warnings_as_error == null)
+				warnings_as_error = new List<int> ();
+			
+			warnings_as_error.Add (id);
+		}
 
-				StringBuilder msg = new StringBuilder ();
-				if (!location.IsNull) {
-					msg.Append (location.ToString ());
-					msg.Append (" ");
-				}
-				msg.AppendFormat ("{0} CS{1:0000}: {2}", MessageType, code, message);
+		public void RemoveWarningAsError (string warningId)
+		{
+			int id;
+			try {
+				id = int.Parse (warningId);
+			} catch {
+				id = -1;
+			}
 
-				//
-				// 
-				if (Stderr == Console.Error)
-					Stderr.WriteLine (ColorFormat (msg.ToString ()));
-				else
-					Stderr.WriteLine (msg.ToString ());
+			if (!CheckWarningCode (id, warningId, Location.Null))
+				return;
 
-				if (extra_info != null) {
-					foreach (string s in extra_info)
-						Stderr.WriteLine (s + MessageType + ")");
-				}
+			if (warnings_only == null)
+				warnings_only = new List<int> ();
 
-				if (Stacktrace)
-					Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
+			warnings_only.Add (id);
+		}
 
-				if (Fatal && !IsWarning)
-					throw new Exception (message);
+		public bool CheckWarningCode (int code, Location loc)
+		{
+			return CheckWarningCode (code, code.ToString (), loc);
+		}
 
-				Check (code);
-			}
+		public bool CheckWarningCode (int code, string scode, Location loc)
+		{
+			if (IsValidWarning (code))
+				return true;
 
-			protected virtual string ColorFormat (string s)
-			{
-				return s;
-			}
+			Warning (1691, 1, loc, "`{0}' is not a valid warning number", scode);
+			return false;
 		}
 
-		sealed class WarningMessage : AbstractMessage
+		public void ExtraInformation (Location loc, string msg)
 		{
-			readonly int Level;
+			extra_information.Add (String.Format ("{0} {1}", loc, msg));
+		}
 
-			public WarningMessage (int code, int level, Location loc, string message, ArrayList extra_info)
-				: base (code, loc, message, extra_info)
-			{
-				Level = level;
+		public WarningRegions RegisterWarningRegion (Location location)
+		{
+			WarningRegions regions;
+			if (warning_regions_table == null) {
+				regions = null;
+				warning_regions_table = new Dictionary<string, WarningRegions> ();
+			} else {
+				warning_regions_table.TryGetValue (location.Name, out regions);
 			}
 
-			public override bool IsWarning {
-				get { return true; }
+			if (regions == null) {
+				regions = new WarningRegions ();
+				warning_regions_table.Add (location.Name, regions);
 			}
 
-			bool IsEnabled ()
-			{
-				if (WarningLevel < Level)
-					return false;
-
-				if (warning_ignore_table != null) {
-					if (warning_ignore_table.Contains (code)) {
-						return false;
-					}
-				}
-
-				if (warning_regions_table == null || location.IsNull)
-					return true;
+			return regions;
+		}
 
-				WarningRegions regions = (WarningRegions)warning_regions_table [location.Name];
-				if (regions == null)
-					return true;
+		public void Warning (int code, int level, Location loc, string message)
+		{
+			if (reporting_disabled > 0)
+				return;
 
-				return regions.IsWarningEnabled (code, location.Row);
-			}
+			if (!IsWarningEnabled (code, level, loc))
+				return;
 
-			bool IsErrorWarning {
-				get {
-					bool is_error = WarningsAreErrors;
+			AbstractMessage msg;
+			if (IsWarningAsError (code))
+				msg = new ErrorMessage (code, loc, message, extra_information);
+			else
+				msg = new WarningMessage (code, loc, message, extra_information);
 
-					// Check specific list
-					if (warnings_as_error != null)
-						is_error |= warnings_as_error.Contains (code);
+			extra_information.Clear ();
+			printer.Print (msg);
+		}
 
-					// Ignore excluded warnings
-					if (warnings_only != null && warnings_only.Contains (code))
-						is_error = false;
+		public void Warning (int code, int level, Location loc, string format, string arg)
+		{
+			Warning (code, level, loc, String.Format (format, arg));
+		}
 
-					return is_error;
-				}
-			}
+		public void Warning (int code, int level, Location loc, string format, string arg1, string arg2)
+		{
+			Warning (code, level, loc, String.Format (format, arg1, arg2));
+		}
 
-			public override void Print ()
-			{
-				if (!IsEnabled ())
-					return;
+		public void Warning (int code, int level, Location loc, string format, params object[] args)
+		{
+			Warning (code, level, loc, String.Format (format, args));
+		}
 
-				if (IsErrorWarning) {
-					new ErrorMessage (this).Print ();
-					return;
-				}
+		public void Warning (int code, int level, string message)
+		{
+			Warning (code, level, Location.Null, message);
+		}
 
-				Warnings++;
-				base.Print ();
-			}
+		public void Warning (int code, int level, string format, string arg)
+		{
+			Warning (code, level, Location.Null, format, arg);
+		}
 
-			public override string MessageType {
-				get {
-					return "warning";
-				}
-			}
+		public void Warning (int code, int level, string format, string arg1, string arg2)
+		{
+			Warning (code, level, Location.Null, format, arg1, arg2);
 		}
 
-		static int NameToCode (string s)
+		public void Warning (int code, int level, string format, params string[] args)
 		{
-			switch (s){
-			case "black":
-				return 0;
-			case "red":
-				return 1;
-			case "green":
-				return 2;
-			case "yellow":
-				return 3;
-			case "blue":
-				return 4;
-			case "magenta":
-				return 5;
-			case "cyan":
-				return 6;
-			case "grey":
-			case "white":
-				return 7;
-			}
-			return 7;
+			Warning (code, level, Location.Null, String.Format (format, args));
 		}
-		
+
 		//
-		// maps a color name to its xterm color code
+		// Warnings encountered so far
 		//
-		static string GetForeground (string s)
+		public int Warnings {
+			get { return printer.WarningsCount; }
+		}
+
+		public void Error (int code, Location loc, string error)
 		{
-			string highcode;
+			if (reporting_disabled > 0)
+				return;
 
-			if (s.StartsWith ("bright")){
-				highcode = "1;";
-				s = s.Substring (6);
-			} else
-				highcode = "";
+			ErrorMessage msg = new ErrorMessage (code, loc, error, extra_information);
+			extra_information.Clear ();
 
-			return "\x001b[" + highcode + (30 + NameToCode (s)).ToString () + "m";
+			printer.Print (msg);
 		}
 
-		static string GetBackground (string s)
+		public void Error (int code, Location loc, string format, string arg)
 		{
-			return "\x001b[" + (40 + NameToCode (s)).ToString () + "m";
+			Error (code, loc, String.Format (format, arg));
 		}
-		
-		sealed class ErrorMessage : AbstractMessage
-		{
-			static string prefix, postfix;
 
-			static ErrorMessage ()
-			{
-				string term = Environment.GetEnvironmentVariable ("TERM");
-				bool xterm_colors = false;
-				
-				switch (term){
-				case "xterm":
-				case "rxvt":
-				case "rxvt-unicode": 
-					if (Environment.GetEnvironmentVariable ("COLORTERM") != null){
-						xterm_colors = true;
-					}
-					break;
+		public void Error (int code, Location loc, string format, string arg1, string arg2)
+		{
+			Error (code, loc, String.Format (format, arg1, arg2));
+		}
 
-				case "xterm-color":
-					xterm_colors = true;
-					break;
-				}
-				if (!xterm_colors)
-					return;
-
-				if (!(UnixUtils.isatty (1) && UnixUtils.isatty (2)))
-					return;
-				
-				string config = Environment.GetEnvironmentVariable ("MCS_COLORS");
-				if (config == null){
-					config = "errors=red";
-					//config = "brightwhite,red";
-				}
-
-				if (config == "disable")
-					return;
-
-				if (!config.StartsWith ("errors="))
-					return;
-
-				config = config.Substring (7);
-				
-				int p = config.IndexOf (",");
-				if (p == -1)
-					prefix = GetForeground (config);
-				else
-					prefix = GetBackground (config.Substring (p+1)) + GetForeground (config.Substring (0, p));
-				postfix = "\x001b[0m";
-			}
-
-			public ErrorMessage (int code, Location loc, string message, ArrayList extraInfo)
-				: base (code, loc, message, extraInfo)
-			{
-			}
-
-			public ErrorMessage (AbstractMessage aMsg)
-				: base (aMsg)
-			{
-			}
-
-			protected override string ColorFormat (string s)
-			{
-				if (prefix != null)
-					return prefix + s + postfix;
-				return s;
-			}
-			
-			public override void Print()
-			{
-				Errors++;
-				base.Print ();
-			}
-
-			public override bool IsWarning {
-				get { return false; }
-			}
-
-			public override string MessageType {
-				get {
-					return "error";
-				}
-			}
+		public void Error (int code, Location loc, string format, params string[] args)
+		{
+			Error (code, loc, String.Format (format, args));
 		}
 
-		public static void FeatureIsNotAvailable (Location loc, string feature)
+		public void Error (int code, string error)
 		{
-			string version;
-			switch (RootContext.Version) {
-			case LanguageVersion.ISO_1:
-				version = "1.0";
-				break;
-			case LanguageVersion.ISO_2:
-				version = "2.0";
-				break;
-			case LanguageVersion.V_3:
-				version = "3.0";
-				break;
-			default:
-				throw new InternalErrorException ("Invalid feature version", RootContext.Version);
-			}
-
-			Report.Error (1644, loc,
-				"Feature `{0}' cannot be used because it is not part of the C# {1} language specification",
-				      feature, version);
+			Error (code, Location.Null, error);
 		}
 
-		public static void FeatureIsNotSupported (Location loc, string feature)
+		public void Error (int code, string format, string arg)
 		{
-			Report.Error (1644, loc,
-				"Feature `{0}' is not supported in Mono mcs1 compiler. Consider using the `gmcs' compiler instead",
-				feature);
+			Error (code, Location.Null, format, arg);
 		}
-		
-		public static string FriendlyStackTrace (Exception e)
+
+		public void Error (int code, string format, string arg1, string arg2)
 		{
-			return FriendlyStackTrace (new StackTrace (e, true));
-		}
-		
-		static string FriendlyStackTrace (StackTrace t)
-		{		
-			StringBuilder sb = new StringBuilder ();
-			
-			bool foundUserCode = false;
-			
-			for (int i = 0; i < t.FrameCount; i++) {
-				StackFrame f = t.GetFrame (i);
-				MethodBase mb = f.GetMethod ();
-				
-				if (!foundUserCode && mb.ReflectedType == typeof (Report))
-					continue;
-				
-				foundUserCode = true;
-				
-				sb.Append ("\tin ");
-				
-				if (f.GetFileLineNumber () > 0)
-					sb.AppendFormat ("(at {0}:{1}) ", f.GetFileName (), f.GetFileLineNumber ());
-				
-				sb.AppendFormat ("{0}.{1} (", mb.ReflectedType.Name, mb.Name);
-				
-				bool first = true;
-				foreach (ParameterInfo pi in mb.GetParameters ()) {
-					if (!first)
-						sb.Append (", ");
-					first = false;
-					
-					sb.Append (TypeManager.CSharpName (pi.ParameterType));
-				}
-				sb.Append (")\n");
-			}
-	
-			return sb.ToString ();
+			Error (code, Location.Null, format, arg1, arg2);
 		}
 
-		public static void StackTrace ()
+		public void Error (int code, string format, params string[] args)
 		{
-			Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
+			Error (code, Location.Null, String.Format (format, args));
 		}
 
-		static bool IsValidWarning (int code)
-		{	
-			return Array.BinarySearch (AllWarnings, code) >= 0;
+		//
+		// Errors encountered so far
+		//
+		public int Errors {
+			get { return printer.ErrorsCount; }
 		}
-		        
-		static public void RuntimeMissingSupport (Location loc, string feature) 
-		{
-			Report.Error (-88, loc, "Your .NET Runtime does not support `{0}'. Please use the latest Mono runtime instead.", feature);
+
+		public ReportPrinter Printer {
+			get { return printer; }
 		}
 
-		/// <summary>
-		/// In most error cases is very useful to have information about symbol that caused the error.
-		/// Call this method before you call Report.Error when it makes sense.
-		/// </summary>
-		static public void SymbolRelatedToPreviousError (Location loc, string symbol)
+		public void SetIgnoreWarning (int code)
 		{
-			SymbolRelatedToPreviousError (loc.ToString (), symbol);
+			if (warning_ignore_table == null)
+				warning_ignore_table = new Dictionary<int, bool> ();
+
+			warning_ignore_table [code] = true;
 		}
 
-		static public void SymbolRelatedToPreviousError (MemberInfo mi)
+		public ReportPrinter SetPrinter (ReportPrinter printer)
 		{
-			if (reporting_disabled)
-				return;
-
-			Type dt = TypeManager.DropGenericTypeArguments (mi.DeclaringType);
-			if (TypeManager.IsDelegateType (dt)) {
-				SymbolRelatedToPreviousError (dt);
-				return;
-			}			
-			
-			DeclSpace temp_ds = TypeManager.LookupDeclSpace (dt);
-			if (temp_ds == null) {
-				SymbolRelatedToPreviousError (dt.Assembly.Location, TypeManager.GetFullNameSignature (mi));
-			} else {
-				MethodBase mb = mi as MethodBase;
-				if (mb != null) {
-					mb = TypeManager.DropGenericMethodArguments (mb);
-					IMethodData md = TypeManager.GetMethod (mb);
-					if (md != null)
-						SymbolRelatedToPreviousError (md.Location, md.GetSignatureForError ());
-
-					return;
-				}
+			ReportPrinter old = this.printer;
+			this.printer = printer;
+			return old;
+		}
 
-				// FIXME: Completely wrong, it has to use FindMembers
-				MemberCore mc = temp_ds.GetDefinition (mi.Name);
-				if (mc != null)
-					SymbolRelatedToPreviousError (mc);
+		public int WarningLevel {
+			get {
+				return warning_level;
+			}
+			set {
+				warning_level = value;
 			}
 		}
 
-		static public void SymbolRelatedToPreviousError (MemberCore mc)
+		[Conditional ("MCS_DEBUG")]
+		static public void Debug (string message, params object[] args)
 		{
-			SymbolRelatedToPreviousError (mc.Location, mc.GetSignatureForError ());
+			Debug (4, message, args);
 		}
-
-		static public void SymbolRelatedToPreviousError (Type type)
+			
+		[Conditional ("MCS_DEBUG")]
+		static public void Debug (int category, string message, params object[] args)
 		{
-			if (reporting_disabled)
+			if ((category & DebugFlags) == 0)
 				return;
 
-			type = TypeManager.DropGenericTypeArguments (type);
+			StringBuilder sb = new StringBuilder (message);
 
-			if (TypeManager.IsGenericParameter (type)) {
-				TypeParameter tp = TypeManager.LookupTypeParameter (type);
-				if (tp != null) {
-					SymbolRelatedToPreviousError (tp.Location, "");
-					return;
+			if ((args != null) && (args.Length > 0)) {
+				sb.Append (": ");
+
+				bool first = true;
+				foreach (object arg in args) {
+					if (first)
+						first = false;
+					else
+						sb.Append (", ");
+					if (arg == null)
+						sb.Append ("null");
+//					else if (arg is ICollection)
+//						sb.Append (PrintCollection ((ICollection) arg));
+					else
+						sb.Append (arg);
 				}
 			}
 
-			if (type is TypeBuilder) {
-				DeclSpace temp_ds = TypeManager.LookupDeclSpace (type);
-				SymbolRelatedToPreviousError (temp_ds.Location, TypeManager.CSharpName (type));
-			} else if (TypeManager.HasElementType (type)) {
-				SymbolRelatedToPreviousError (TypeManager.GetElementType (type));
-			} else {
-				SymbolRelatedToPreviousError (type.Assembly.Location, TypeManager.CSharpName (type));
+			Console.WriteLine (sb.ToString ());
+		}
+/*
+		static public string PrintCollection (ICollection collection)
+		{
+			StringBuilder sb = new StringBuilder ();
+
+			sb.Append (collection.GetType ());
+			sb.Append ("(");
+
+			bool first = true;
+			foreach (object o in collection) {
+				if (first)
+					first = false;
+				else
+					sb.Append (", ");
+				sb.Append (o);
 			}
+
+			sb.Append (")");
+			return sb.ToString ();
 		}
+*/ 
+	}
+
+	public abstract class AbstractMessage
+	{
+		readonly string[] extra_info;
+		protected readonly int code;
+		protected readonly Location location;
+		readonly string message;
 
-		static void SymbolRelatedToPreviousError (string loc, string symbol)
+		protected AbstractMessage (int code, Location loc, string msg, List<string> extraInfo)
 		{
-			string msg = String.Format ("{0} (Location of the symbol related to previous ", loc);
-			if (extra_information.Contains (msg))
-				return;
+			this.code = code;
+			if (code < 0)
+				this.code = 8000 - code;
 
-			extra_information.Add (msg);
+			this.location = loc;
+			this.message = msg;
+			if (extraInfo.Count != 0) {
+				this.extra_info = extraInfo.ToArray ();
+			}
 		}
 
-		public static void AddWarningAsError (string warningId)
+		protected AbstractMessage (AbstractMessage aMsg)
 		{
-			int id;
-			try {
-				id = int.Parse (warningId);
-			} catch {
-				id = -1;
-			}
+			this.code = aMsg.code;
+			this.location = aMsg.location;
+			this.message = aMsg.message;
+			this.extra_info = aMsg.extra_info;
+		}
 
-			if (!CheckWarningCode (id, warningId, Location.Null))
-				return;
+		public int Code {
+			get { return code; }
+		}
 
-			if (warnings_as_error == null)
-				warnings_as_error = new ArrayList ();
-			
-			warnings_as_error.Add (id);
+		public override bool Equals (object obj)
+		{
+			AbstractMessage msg = obj as AbstractMessage;
+			if (msg == null)
+				return false;
+
+			return code == msg.code && location.Equals (msg.location) && message == msg.message;
 		}
 
-		public static void RemoveWarningAsError (string warningId)
+		public override int GetHashCode ()
 		{
-			int id;
-			try {
-				id = int.Parse (warningId);
-			} catch {
-				id = -1;
-			}
+			return code.GetHashCode ();
+		}
 
-			if (!CheckWarningCode (id, warningId, Location.Null))
-				return;
+		public abstract bool IsWarning { get; }
 
-			if (warnings_only == null)
-				warnings_only = new ArrayList ();
+		public Location Location {
+			get { return location; }
+		}
 
-			warnings_only.Add (id);
+		public abstract string MessageType { get; }
+
+		public string[] RelatedSymbols {
+			get { return extra_info; }
 		}
 
-		public static bool CheckWarningCode (int code, Location loc)
-		{
-			return CheckWarningCode (code, code.ToString (), loc);
+		public string Text {
+			get { return message; }
 		}
+	}
 
-		public static bool CheckWarningCode (int code, string scode, Location loc)
+	sealed class WarningMessage : AbstractMessage
+	{
+		public WarningMessage (int code, Location loc, string message, List<string> extra_info)
+			: base (code, loc, message, extra_info)
 		{
-			if (IsValidWarning (code))
-				return true;
+		}
 
-			Report.Warning (1691, 1, loc, "`{0}' is not a valid warning number", scode);
-			return false;
+		public override bool IsWarning {
+			get { return true; }
 		}
 
-		public static void ExtraInformation (Location loc, string msg)
+		public override string MessageType {
+			get {
+				return "warning";
+			}
+		}
+	}
+
+	sealed class ErrorMessage : AbstractMessage
+	{
+		public ErrorMessage (int code, Location loc, string message, List<string> extraInfo)
+			: base (code, loc, message, extraInfo)
 		{
-			extra_information.Add (String.Format ("{0} {1}", loc, msg));
 		}
 
-		public static WarningRegions RegisterWarningRegion (Location location)
+		public ErrorMessage (AbstractMessage aMsg)
+			: base (aMsg)
 		{
-			if (warning_regions_table == null)
-				warning_regions_table = new Hashtable ();
+		}
 
-			WarningRegions regions = (WarningRegions)warning_regions_table [location.Name];
-			if (regions == null) {
-				regions = new WarningRegions ();
-				warning_regions_table.Add (location.Name, regions);
-			}
-			return regions;
+		public override bool IsWarning {
+			get { return false; }
 		}
 
-		static public void Warning (int code, int level, Location loc, string message)
-		{
-			WarningMessage w = new WarningMessage (code, level, loc, message, extra_information);
-			extra_information.Clear ();
-			w.Print ();
+		public override string MessageType {
+			get {
+				return "error";
+			}
 		}
+	}
 
-		static public void Warning (int code, int level, Location loc, string format, string arg)
-		{
-			WarningMessage w = new WarningMessage (code, level, loc, String.Format (format, arg), extra_information);
-			extra_information.Clear ();
-			w.Print ();
+	//
+	// Generic base for any message writer
+	//
+	public abstract class ReportPrinter {
+		/// <summary>  
+		///   Whether to dump a stack trace on errors. 
+		/// </summary>
+		public bool Stacktrace;
+		
+		int warnings, errors;
+
+		public int WarningsCount {
+			get { return warnings; }
+		}
+		
+		public int ErrorsCount {
+			get { return errors; }
 		}
 
-		static public void Warning (int code, int level, Location loc, string format, string arg1, string arg2)
+		protected virtual string FormatText (string txt)
 		{
-			WarningMessage w = new WarningMessage (code, level, loc, String.Format (format, arg1, arg2), extra_information);
-			extra_information.Clear ();
-			w.Print ();
+			return txt;
 		}
 
-		static public void Warning (int code, int level, Location loc, string format, params object[] args)
-		{
-			WarningMessage w = new WarningMessage (code, level, loc, String.Format (format, args), extra_information);
-			extra_information.Clear ();
-			w.Print ();
+		//
+		// When (symbols related to previous ...) can be used
+		//
+		public virtual bool HasRelatedSymbolSupport {
+			get { return true; }
 		}
 
-		static public void Warning (int code, int level, string message)
+		public virtual void Print (AbstractMessage msg)
 		{
-			Warning (code, level, Location.Null, message);
+			if (msg.IsWarning)
+				++warnings;
+			else
+				++errors;
 		}
 
-		static public void Warning (int code, int level, string format, string arg)
+		protected void Print (AbstractMessage msg, TextWriter output)
 		{
-			Warning (code, level, Location.Null, format, arg);
+			StringBuilder txt = new StringBuilder ();
+			if (!msg.Location.IsNull) {
+				txt.Append (msg.Location.ToString ());
+				txt.Append (" ");
+			}
+
+			txt.AppendFormat ("{0} CS{1:0000}: {2}", msg.MessageType, msg.Code, msg.Text);
+
+			if (!msg.IsWarning)
+				output.WriteLine (FormatText (txt.ToString ()));
+			else
+				output.WriteLine (txt.ToString ());
+
+			if (msg.RelatedSymbols != null) {
+				foreach (string s in msg.RelatedSymbols)
+					output.WriteLine (s + msg.MessageType + ")");
+			}
 		}
+	}
 
-		static public void Warning (int code, int level, string format, string arg1, string arg2)
+	//
+	// Default message recorder, it uses two types of message groups.
+	// Common messages: messages reported in all sessions.
+	// Merged messages: union of all messages in all sessions. 
+	//
+	// Used by the Lambda expressions to compile the code with various
+	// parameter values, or by attribute resolver
+	//
+	class SessionReportPrinter : ReportPrinter
+	{
+		List<AbstractMessage> session_messages;
+		//
+		// A collection of exactly same messages reported in all sessions
+		//
+		List<AbstractMessage> common_messages;
+
+		//
+		// A collection of unique messages reported in all sessions
+		//
+		List<AbstractMessage> merged_messages;
+
+		public override void Print (AbstractMessage msg)
 		{
-			Warning (code, level, Location.Null, format, arg1, arg2);
+			//
+			// This line is useful when debugging recorded messages
+			//
+			// Console.WriteLine ("RECORDING: {0} {1} {2}", code, location, message);
+
+			if (session_messages == null)
+				session_messages = new List<AbstractMessage> ();
+
+			session_messages.Add (msg);
+
+			base.Print (msg);
 		}
 
-		static public void Warning (int code, int level, string format, params string[] args)
+		public void EndSession ()
 		{
-			Warning (code, level, Location.Null, String.Format (format, args));
+			if (session_messages == null)
+				return;
+
+			//
+			// Handles the first session
+			//
+			if (common_messages == null) {
+				common_messages = new List<AbstractMessage> (session_messages);
+				merged_messages = session_messages;
+				session_messages = null;
+				return;
+			}
+
+			//
+			// Store common messages if any
+			//
+			for (int i = 0; i < common_messages.Count; ++i) {
+				AbstractMessage cmsg = (AbstractMessage) common_messages[i];
+				bool common_msg_found = false;
+				foreach (AbstractMessage msg in session_messages) {
+					if (cmsg.Equals (msg)) {
+						common_msg_found = true;
+						break;
+					}
+				}
+
+				if (!common_msg_found)
+					common_messages.RemoveAt (i);
+			}
+
+			//
+			// Merge session and previous messages
+			//
+			for (int i = 0; i < session_messages.Count; ++i) {
+				AbstractMessage msg = (AbstractMessage) session_messages[i];
+				bool msg_found = false;
+				for (int ii = 0; ii < merged_messages.Count; ++ii) {
+					if (msg.Equals (merged_messages[ii])) {
+						msg_found = true;
+						break;
+					}
+				}
+
+				if (!msg_found)
+					merged_messages.Add (msg);
+			}
 		}
 
-		static public void Error (int code, Location loc, string error)
-		{
-			new ErrorMessage (code, loc, error, extra_information).Print ();
-			extra_information.Clear ();
+		public bool IsEmpty {
+			get {
+				return merged_messages == null && common_messages == null;
+			}
 		}
 
-		static public void Error (int code, Location loc, string format, string arg)
+		//
+		// Prints collected messages, common messages have a priority
+		//
+		public bool Merge (ReportPrinter dest)
 		{
-			new ErrorMessage (code, loc, String.Format (format, arg), extra_information).Print ();
-			extra_information.Clear ();
+			var messages_to_print = merged_messages;
+			if (common_messages != null && common_messages.Count > 0) {
+				messages_to_print = common_messages;
+			}
+
+			if (messages_to_print == null)
+				return false;
+
+			foreach (AbstractMessage msg in messages_to_print)
+				dest.Print (msg);
+
+			return true;
 		}
+	}
+
+	class StreamReportPrinter : ReportPrinter
+	{
+		readonly TextWriter writer;
 
-		static public void Error (int code, Location loc, string format, string arg1, string arg2)
+		public StreamReportPrinter (TextWriter writer)
 		{
-			new ErrorMessage (code, loc, String.Format (format, arg1, arg2), extra_information).Print ();
-			extra_information.Clear ();
+			this.writer = writer;
 		}
 
-		static public void Error (int code, Location loc, string format, params object[] args)
+		public override void Print (AbstractMessage msg)
 		{
-			Error (code, loc, String.Format (format, args));
+			Print (msg, writer);
+			base.Print (msg);
 		}
+	}
 
-		static public void Error (int code, string error)
+	class ConsoleReportPrinter : StreamReportPrinter
+	{
+		static readonly string prefix, postfix;
+
+		static ConsoleReportPrinter ()
 		{
-			Error (code, Location.Null, error);
+			string term = Environment.GetEnvironmentVariable ("TERM");
+			bool xterm_colors = false;
+			
+			switch (term){
+			case "xterm":
+			case "rxvt":
+			case "rxvt-unicode": 
+				if (Environment.GetEnvironmentVariable ("COLORTERM") != null){
+					xterm_colors = true;
+				}
+				break;
+
+			case "xterm-color":
+				xterm_colors = true;
+				break;
+			}
+			if (!xterm_colors)
+				return;
+
+			if (!(UnixUtils.isatty (1) && UnixUtils.isatty (2)))
+				return;
+			
+			string config = Environment.GetEnvironmentVariable ("MCS_COLORS");
+			if (config == null){
+				config = "errors=red";
+				//config = "brightwhite,red";
+			}
+
+			if (config == "disable")
+				return;
+
+			if (!config.StartsWith ("errors="))
+				return;
+
+			config = config.Substring (7);
+			
+			int p = config.IndexOf (",");
+			if (p == -1)
+				prefix = GetForeground (config);
+			else
+				prefix = GetBackground (config.Substring (p+1)) + GetForeground (config.Substring (0, p));
+			postfix = "\x001b[0m";
 		}
 
-		static public void Error (int code, string format, string arg)
+		public ConsoleReportPrinter ()
+			: base (Console.Error)
 		{
-			Error (code, Location.Null, format, arg);
 		}
 
-		static public void Error (int code, string format, string arg1, string arg2)
+		public ConsoleReportPrinter (TextWriter writer)
+			: base (writer)
 		{
-			Error (code, Location.Null, format, arg1, arg2);
 		}
 
-		static public void Error (int code, string format, params string[] args)
+		public int Fatal { get; set; }
+
+		static int NameToCode (string s)
 		{
-			Error (code, Location.Null, String.Format (format, args));
+			switch (s) {
+			case "black":
+				return 0;
+			case "red":
+				return 1;
+			case "green":
+				return 2;
+			case "yellow":
+				return 3;
+			case "blue":
+				return 4;
+			case "magenta":
+				return 5;
+			case "cyan":
+				return 6;
+			case "grey":
+			case "white":
+				return 7;
+			}
+			return 7;
 		}
 
-		static public void SetIgnoreWarning (int code)
+		//
+		// maps a color name to its xterm color code
+		//
+		static string GetForeground (string s)
 		{
-			if (warning_ignore_table == null)
-				warning_ignore_table = new Hashtable ();
+			string highcode;
 
-			warning_ignore_table [code] = true;
-		}
-		
-		static public int ExpectedError {
-			set {
-				expected_error = value;
-			}
-			get {
-				return expected_error;
-			}
-		}
-		
-		public static int WarningLevel {
-			get {
-				return warning_level;
-			}
-			set {
-				warning_level = value;
-			}
-		}
+			if (s.StartsWith ("bright")) {
+				highcode = "1;";
+				s = s.Substring (6);
+			} else
+				highcode = "";
 
-		public static int DebugFlags = 0;
+			return "\x001b[" + highcode + (30 + NameToCode (s)).ToString () + "m";
+		}
 
-		[Conditional ("MCS_DEBUG")]
-		static public void Debug (string message, params object[] args)
+		static string GetBackground (string s)
 		{
-			Debug (4, message, args);
+			return "\x001b[" + (40 + NameToCode (s)).ToString () + "m";
 		}
-			
-		[Conditional ("MCS_DEBUG")]
-		static public void Debug (int category, string message, params object[] args)
-		{
-			if ((category & DebugFlags) == 0)
-				return;
 
-			StringBuilder sb = new StringBuilder (message);
+		protected override string FormatText (string txt)
+		{
+			if (prefix != null)
+				return prefix + txt + postfix;
 
-			if ((args != null) && (args.Length > 0)) {
-				sb.Append (": ");
+			return txt;
+		}
 
+		static string FriendlyStackTrace (StackTrace t)
+		{		
+			StringBuilder sb = new StringBuilder ();
+			
+			bool foundUserCode = false;
+			
+			for (int i = 0; i < t.FrameCount; i++) {
+				StackFrame f = t.GetFrame (i);
+				MethodBase mb = f.GetMethod ();
+				
+				if (!foundUserCode && mb.ReflectedType == typeof (Report))
+					continue;
+				
+				foundUserCode = true;
+				
+				sb.Append ("\tin ");
+				
+				if (f.GetFileLineNumber () > 0)
+					sb.AppendFormat ("(at {0}:{1}) ", f.GetFileName (), f.GetFileLineNumber ());
+				
+				sb.AppendFormat ("{0}.{1} (", mb.ReflectedType.Name, mb.Name);
+				
 				bool first = true;
-				foreach (object arg in args) {
-					if (first)
-						first = false;
-					else
+				foreach (ParameterInfo pi in mb.GetParameters ()) {
+					if (!first)
 						sb.Append (", ");
-					if (arg == null)
-						sb.Append ("null");
-					else if (arg is ICollection)
-						sb.Append (PrintCollection ((ICollection) arg));
-					else
-						sb.Append (arg);
+					first = false;
+
+					sb.Append (pi.ParameterType.FullName);
 				}
+				sb.Append (")\n");
 			}
-
-			Console.WriteLine (sb.ToString ());
+	
+			return sb.ToString ();
 		}
 
-		static public string PrintCollection (ICollection collection)
+		int print_count;
+		public override void Print (AbstractMessage msg)
 		{
-			StringBuilder sb = new StringBuilder ();
+			base.Print (msg);
 
-			sb.Append (collection.GetType ());
-			sb.Append ("(");
+			if (Stacktrace)
+				Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
 
-			bool first = true;
-			foreach (object o in collection) {
-				if (first)
-					first = false;
-				else
-					sb.Append (", ");
-				sb.Append (o);
-			}
+			if (++print_count == Fatal)
+				throw new Exception (msg.Text);
+		}
 
-			sb.Append (")");
-			return sb.ToString ();
+		public static string FriendlyStackTrace (Exception e)
+		{
+			return FriendlyStackTrace (new StackTrace (e, true));
+		}
+
+		public static void StackTrace ()
+		{
+			Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
 		}
 	}
 
@@ -1167,14 +1127,14 @@ namespace Mono.CSharp {
 		}
 
 
-		ArrayList regions = new ArrayList ();
+		List<PragmaCmd> regions = new List<PragmaCmd> ();
 
 		public void WarningDisable (int line)
 		{
 			regions.Add (new DisableAll (line));
 		}
 
-		public void WarningDisable (Location location, int code)
+		public void WarningDisable (Location location, int code, Report Report)
 		{
 			if (Report.CheckWarningCode (code, location))
 				regions.Add (new Disable (location.Row, code));
@@ -1185,7 +1145,7 @@ namespace Mono.CSharp {
 			regions.Add (new EnableAll (line));
 		}
 
-		public void WarningEnable (Location location, int code)
+		public void WarningEnable (Location location, int code, Report Report)
 		{
 			if (Report.CheckWarningCode (code, location))
 				regions.Add (new Enable (location.Row, code));