// report.cs: report errors and warnings.
//
// Author: Miguel de Icaza (miguel@ximian.com)
+// Marek Safar (marek.safar@seznam.cz)
//
// (C) 2001 Ximian, Inc. (http://www.ximian.com)
//
-//
-// FIXME: currently our class library does not support custom number format strings
-//
using System;
+using System.IO;
using System.Text;
using System.Collections;
using System.Collections.Specialized;
using System.Diagnostics;
using System.Reflection;
+using System.Reflection.Emit;
namespace Mono.CSharp {
/// Whether to dump a stack trace on errors.
/// </summary>
static public bool Stacktrace;
+
+ static public TextWriter Stderr = Console.Error;
//
// If the 'expected' error code is reported then the
//
// Keeps track of the warnings that we are ignoring
//
- static Hashtable warning_ignore_table;
+ public static Hashtable warning_ignore_table;
+
+ static Hashtable warning_regions_table;
/// <summary>
/// List of symbols related to reported error/warning. You have to fill it before error/warning is reported.
/// </summary>
- static StringCollection related_symbols = new StringCollection ();
+ static StringCollection extra_information = new StringCollection ();
- static void Check (int code)
- {
- if (code == expected_error){
- if (Fatal)
- throw new Exception ();
-
- Environment.Exit (0);
+ //
+ // IF YOU ADD A NEW WARNING YOU HAVE TO ADD ITS ID HERE
+ //
+ public static readonly int[] AllWarnings = new int[] {
+ 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,
+ 612, 618, 626, 628, 642, 649, 652, 658, 659, 660, 661, 665, 672, 675,
+ 809,
+ 1030,
+ 1522, 1570, 1571, 1572, 1573, 1574, 1580, 1581, 1584, 1587, 1589, 1590, 1591, 1592,
+ 1616, 1633, 1634, 1635, 1690, 1691, 1692,
+ 1717, 1718, 1720,
+ 1901,
+ 2002, 2023,
+ 3005, 3012, 3018, 3019, 3021, 3022, 3023, 3026, 3027,
+#if GMCS_SOURCE
+ 402, 414, 693, 1058, 1700, 3024
+#endif
+ };
+
+ static Report ()
+ {
+ // Just to be sure that binary search is working
+ Array.Sort (AllWarnings);
+ }
+
+ public static void Reset ()
+ {
+ Errors = Warnings = 0;
+ WarningsAreErrors = false;
+ warning_ignore_table = null;
+ warning_regions_table = null;
+ }
+
+ abstract class AbstractMessage {
+
+ static void Check (int code)
+ {
+ if (code == expected_error) {
+ Environment.Exit (0);
+ }
+ }
+
+ public abstract bool IsWarning { get; }
+
+ public abstract string MessageType { get; }
+
+ public virtual void Print (int code, string location, string text)
+ {
+ if (code < 0)
+ code = 8000-code;
+
+ StringBuilder msg = new StringBuilder ();
+ if (location.Length != 0) {
+ msg.Append (location);
+ msg.Append (' ');
+ }
+ msg.AppendFormat ("{0} CS{1:0000}: {2}", MessageType, code, text);
+ Stderr.WriteLine (msg.ToString ());
+
+ foreach (string s in extra_information)
+ Stderr.WriteLine (s + MessageType + ")");
+
+ extra_information.Clear ();
+
+ if (Stacktrace)
+ Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
+
+ if (Fatal) {
+ if (!IsWarning || WarningsAreErrors)
+ throw new Exception (text);
+ }
+
+ Check (code);
+ }
+
+ public virtual void Print (int code, Location location, string text)
+ {
+ Print (code, location.IsNull ? "" : location.ToString (), text);
+ }
+ }
+
+ sealed class WarningMessage : AbstractMessage {
+ Location loc = Location.Null;
+ readonly int Level;
+
+ public WarningMessage (int level)
+ {
+ Level = level;
+ }
+
+ public override bool IsWarning {
+ get { return true; }
+ }
+
+ bool IsEnabled (int code)
+ {
+ if (RootContext.WarningLevel < Level)
+ return false;
+
+ if (warning_ignore_table != null) {
+ if (warning_ignore_table.Contains (code)) {
+ return false;
+ }
+ }
+
+ if (warning_regions_table == null || loc.Equals (Location.Null))
+ return true;
+
+ WarningRegions regions = (WarningRegions)warning_regions_table [loc.Name];
+ if (regions == null)
+ return true;
+
+ return regions.IsWarningEnabled (code, loc.Row);
+ }
+
+ public override void Print(int code, string location, string text)
+ {
+ if (!IsEnabled (code)) {
+ extra_information.Clear ();
+ return;
+ }
+
+ if (WarningsAreErrors) {
+ new ErrorMessage ().Print (code, location, text);
+ return;
+ }
+
+ Warnings++;
+ base.Print (code, location, text);
+ }
+
+ public override void Print(int code, Location location, string text)
+ {
+ loc = location;
+ base.Print (code, location, text);
+ }
+
+ public override string MessageType {
+ get {
+ return "warning";
+ }
+ }
+ }
+
+ sealed class ErrorMessage : AbstractMessage {
+
+ public override void Print(int code, string location, string text)
+ {
+ Errors++;
+ base.Print (code, location, text);
}
+
+ public override bool IsWarning {
+ get { return false; }
+ }
+
+ public override string MessageType {
+ get {
+ return "error";
+ }
+ }
+
}
- public static void FeatureIsNotStandardized (string feature)
+ public static void FeatureIsNotStandardized (Location loc, string feature)
{
- Report.Error (1644, "Feature '{0}' cannot be used because it is not part of the standardized ISO C# language specification", feature);
+ Report.Error (1644, loc, "Feature `{0}' cannot be used because it is not part of the standardized ISO C# language specification", feature);
}
public static string FriendlyStackTrace (Exception e)
return sb.ToString ();
}
-
- static public void LocationOfPreviousError (Location loc)
+
+ public static void StackTrace ()
{
- Console.WriteLine (String.Format ("{0}({1}) (Location of symbol related to previous error)", loc.Name, loc.Row));
- }
-
- static public void RuntimeMissingSupport (string feature)
+ Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
+ }
+
+ public static bool IsValidWarning (int code)
+ {
+ return Array.BinarySearch (AllWarnings, code) >= 0;
+ }
+
+ static public void RuntimeMissingSupport (Location loc, string feature)
{
- Report.Error (-88, "Your .NET Runtime does not support '{0}'. Please use the latest Mono runtime instead.");
+ Report.Error (-88, loc, "Your .NET Runtime does not support `{0}'. Please use the latest Mono runtime instead.", feature);
}
/// <summary>
/// </summary>
static public void SymbolRelatedToPreviousError (Location loc, string symbol)
{
- SymbolRelatedToPreviousError (String.Format ("{0}({1})", loc.Name, loc.Row), symbol);
+ SymbolRelatedToPreviousError (loc.ToString (), symbol);
}
static public void SymbolRelatedToPreviousError (MemberInfo mi)
{
- TypeContainer temp_ds = TypeManager.LookupTypeContainer (mi.DeclaringType);
+ Type dt = TypeManager.DropGenericTypeArguments (mi.DeclaringType);
+ DeclSpace temp_ds = TypeManager.LookupDeclSpace (dt);
if (temp_ds == null) {
- SymbolRelatedToPreviousError (mi.DeclaringType.Assembly.Location, TypeManager.GetFullNameSignature (mi));
+ SymbolRelatedToPreviousError (dt.Assembly.Location, TypeManager.GetFullNameSignature (mi));
} else {
- if (mi is MethodBase) {
- IMethodData md = TypeManager.GetMethod ((MethodBase)mi);
- SymbolRelatedToPreviousError (md.Location, md.GetSignatureForError (temp_ds));
+ MethodBase mb = mi as MethodBase;
+ if (mb != null) {
+ mb = TypeManager.DropGenericMethodArguments (mb);
+ IMethodData md = TypeManager.GetMethod (mb);
+ SymbolRelatedToPreviousError (md.Location, md.GetSignatureForError ());
return;
}
- string name = String.Concat (temp_ds.Name, ".", mi.Name);
- MemberCore mc = temp_ds.GetDefinition (name) as MemberCore;
- SymbolRelatedToPreviousError (mc.Location, mc.GetSignatureForError ());
+ MemberCore mc = temp_ds.GetDefinition (mi.Name);
+ SymbolRelatedToPreviousError (mc);
}
}
static public void SymbolRelatedToPreviousError (Type type)
{
- DeclSpace temp_ds = TypeManager.LookupDeclSpace (type);
- if (temp_ds == null)
- SymbolRelatedToPreviousError (type.Assembly.Location, TypeManager.CSharpName (type));
- else
+ type = TypeManager.DropGenericTypeArguments (type);
+
+ if (TypeManager.IsGenericParameter (type)) {
+ TypeParameter tp = TypeManager.LookupTypeParameter (type);
+ if (tp != null) {
+ SymbolRelatedToPreviousError (tp.Location, "");
+ return;
+ }
+ }
+
+ if (type is TypeBuilder) {
+ DeclSpace temp_ds = TypeManager.LookupDeclSpace (type);
SymbolRelatedToPreviousError (temp_ds.Location, TypeManager.CSharpName (type));
+ } else if (type.HasElementType) {
+ SymbolRelatedToPreviousError (type.GetElementType ());
+ } else {
+ SymbolRelatedToPreviousError (type.Assembly.Location, TypeManager.CSharpName (type));
+ }
}
static void SymbolRelatedToPreviousError (string loc, string symbol)
{
- related_symbols.Add (String.Format ("{0}: ('{1}' name of symbol related to previous ", loc, symbol));
+ extra_information.Add (String.Format ("{0} (Location of the symbol related to previous ", loc));
}
- static public void RealError (string msg)
+ public static void ExtraInformation (Location loc, string msg)
{
- Errors++;
- Console.WriteLine (msg);
+ extra_information.Add (String.Format ("{0} {1}", loc, msg));
+ }
- foreach (string s in related_symbols)
- Console.WriteLine (s);
- related_symbols.Clear ();
+ public static WarningRegions RegisterWarningRegion (Location location)
+ {
+ if (warning_regions_table == null)
+ warning_regions_table = new Hashtable ();
- if (Stacktrace)
- Console.WriteLine (FriendlyStackTrace (new StackTrace (true)));
-
- if (Fatal)
- throw new Exception (msg);
+ WarningRegions regions = (WarningRegions)warning_regions_table [location.Name];
+ if (regions == null) {
+ regions = new WarningRegions ();
+ warning_regions_table.Add (location.Name, regions);
+ }
+ return regions;
}
+ static public void Warning (int code, int level, Location loc, string message)
+ {
+ WarningMessage w = new WarningMessage (level);
+ w.Print (code, loc, message);
+ }
- static public void Error (int code, Location l, string text)
+ static public void Warning (int code, int level, Location loc, string format, string arg)
{
- if (code < 0)
- code = 8000-code;
-
- string msg = String.Format (
- "{0}({1}) error CS{2:0000}: {3}", l.Name, l.Row, code, text);
-// "{0}({1}) error CS{2}: {3}", l.Name, l.Row, code, text);
-
- RealError (msg);
- Check (code);
+ WarningMessage w = new WarningMessage (level);
+ w.Print (code, loc, String.Format (format, arg));
}
- static public void Warning (int code, Location l, string text)
+ static public void Warning (int code, int level, Location loc, string format, string arg1, string arg2)
{
- if (code < 0)
- code = 8000-code;
-
- if (warning_ignore_table != null){
- if (warning_ignore_table.Contains (code)) {
- related_symbols.Clear ();
- return;
- }
- }
-
- if (WarningsAreErrors)
- Error (code, l, text);
- else {
- string row;
-
- if (Location.IsNull (l))
- row = "";
- else
- row = l.Row.ToString ();
-
- Console.WriteLine (String.Format (
- "{0}({1}) warning CS{2:0000}: {3}",
-// "{0}({1}) warning CS{2}: {3}",
- l.Name, row, code, text));
- Warnings++;
+ WarningMessage w = new WarningMessage (level);
+ w.Print (code, loc, String.Format (format, arg1, arg2));
+ }
- foreach (string s in related_symbols)
- Console.WriteLine (s);
- related_symbols.Clear ();
+ static public void Warning (int code, int level, Location loc, string format, params string[] args)
+ {
+ WarningMessage w = new WarningMessage (level);
+ w.Print (code, loc, String.Format (format, args));
+ }
- Check (code);
+ static public void Warning (int code, int level, string message)
+ {
+ Warning (code, level, Location.Null, message);
+ }
- if (Stacktrace)
- Console.WriteLine (new StackTrace ().ToString ());
- }
+ static public void Warning (int code, int level, string format, string arg)
+ {
+ Warning (code, level, Location.Null, format, arg);
}
-
- static public void Warning (int code, string text)
+
+ static public void Warning (int code, int level, string format, string arg1, string arg2)
{
- Warning (code, Location.Null, text);
+ Warning (code, level, Location.Null, format, arg1, arg2);
}
- static public void Error (int code, string text)
+ static public void Warning (int code, int level, string format, params string[] args)
{
- if (code < 0)
- code = 8000-code;
-
- string msg = String.Format ("error CS{0:0000}: {1}", code, text);
+ Warning (code, level, Location.Null, String.Format (format, args));
+ }
- RealError (msg);
- Check (code);
+ static public void Error (int code, Location loc, string error)
+ {
+ new ErrorMessage ().Print (code, loc, error);
}
- static public void Error (int code, string format, params object[] args)
+ static public void Error (int code, Location loc, string format, string arg)
{
- Error (code, Location.Null, String.Format (format, args));
+ new ErrorMessage ().Print (code, loc, String.Format (format, arg));
+ }
+
+ static public void Error (int code, Location loc, string format, string arg1, string arg2)
+ {
+ new ErrorMessage ().Print (code, loc, String.Format (format, arg1, arg2));
}
- static public void Error (int code, Location loc, string format, params object[] args)
+ static public void Error (int code, Location loc, string format, params string[] args)
{
Error (code, loc, String.Format (format, args));
}
- static public void Warning (int code, string format, params object[] args)
+ static public void Error (int code, string error)
{
- Warning (code, Location.Null, String.Format (format, args));
+ Error (code, Location.Null, error);
}
- static public void Warning (int code, Location loc, string format, params object[] args)
+ static public void Error (int code, string format, string arg)
{
- Warning (code, loc, String.Format (format, args));
+ Error (code, Location.Null, format, arg);
+ }
+
+ static public void Error (int code, string format, string arg1, string arg2)
+ {
+ Error (code, Location.Null, format, arg1, arg2);
+ }
+
+ static public void Error (int code, string format, params string[] args)
+ {
+ Error (code, Location.Null, String.Format (format, args));
}
static public void SetIgnoreWarning (int code)
}
public class InternalErrorException : Exception {
+ public InternalErrorException (Location loc, string text, Exception e)
+ : base (loc + " " + text, e)
+ {
+ }
+
public InternalErrorException ()
: base ("Internal error")
{
: base (message)
{
}
+
+ public InternalErrorException (string message, params object[] args)
+ : base (String.Format (message, args))
+ { }
+ }
+
+ /// <summary>
+ /// Handles #pragma warning
+ /// </summary>
+ public class WarningRegions {
+
+ abstract class PragmaCmd
+ {
+ public int Line;
+
+ protected PragmaCmd (int line)
+ {
+ Line = line;
+ }
+
+ public abstract bool IsEnabled (int code, bool previous);
+ }
+
+ class Disable : PragmaCmd
+ {
+ int code;
+ public Disable (int line, int code)
+ : base (line)
+ {
+ this.code = code;
+ }
+
+ public override bool IsEnabled (int code, bool previous)
+ {
+ return this.code == code ? false : previous;
+ }
+ }
+
+ class DisableAll : PragmaCmd
+ {
+ public DisableAll (int line)
+ : base (line) {}
+
+ public override bool IsEnabled(int code, bool previous)
+ {
+ return false;
+ }
+ }
+
+ class Enable : PragmaCmd
+ {
+ int code;
+ public Enable (int line, int code)
+ : base (line)
+ {
+ this.code = code;
+ }
+
+ public override bool IsEnabled(int code, bool previous)
+ {
+ return this.code == code ? true : previous;
+ }
+ }
+
+ class EnableAll : PragmaCmd
+ {
+ public EnableAll (int line)
+ : base (line) {}
+
+ public override bool IsEnabled(int code, bool previous)
+ {
+ return true;
+ }
+ }
+
+
+ ArrayList regions = new ArrayList ();
+
+ public void WarningDisable (int line)
+ {
+ regions.Add (new DisableAll (line));
+ }
+
+ public void WarningDisable (Location location, int code)
+ {
+ if (CheckWarningCode (code, location))
+ regions.Add (new Disable (location.Row, code));
+ }
+
+ public void WarningEnable (int line)
+ {
+ regions.Add (new EnableAll (line));
+ }
+
+ public void WarningEnable (Location location, int code)
+ {
+ if (CheckWarningCode (code, location))
+ regions.Add (new Enable (location.Row, code));
+ }
+
+ public bool IsWarningEnabled (int code, int src_line)
+ {
+ bool result = true;
+ foreach (PragmaCmd pragma in regions) {
+ if (src_line < pragma.Line)
+ break;
+
+ result = pragma.IsEnabled (code, result);
+ }
+ return result;
+ }
+
+ static bool CheckWarningCode (int code, Location loc)
+ {
+ if (Report.IsValidWarning (code))
+ return true;
+
+ Report.Warning (1691, 1, loc, "`{0}' is not a valid warning number", code.ToString ());
+ return false;
+ }
}
}