Browse Source

Revert "Improved diagnostic output in CheckIgnoredDecls."

This reverts commit 7b1abc79bc.

It breaks the build and it's not trivial to fix it, so given that it's just logging, we'd better disable it for now.

Signed-off-by: Dimitar Dobrev <dpldobrev@protonmail.com>
pull/696/head
Dimitar Dobrev 9 years ago
parent
commit
e68f9bc041
  1. 68
      src/Generator/Passes/CheckIgnoredDecls.cs

68
src/Generator/Passes/CheckIgnoredDecls.cs

@ -56,8 +56,8 @@ namespace CppSharp.Passes
if (!CheckDeclarationAccess(decl)) if (!CheckDeclarationAccess(decl))
{ {
Diagnostics.Debug("'{0}' was ignored due to invalid access", Diagnostics.Debug("Decl '{0}' was ignored due to invalid access",
decl.QualifiedName); decl.Name);
decl.GenerationKind = decl is Field ? GenerationKind.Internal : GenerationKind.None; decl.GenerationKind = decl is Field ? GenerationKind.Internal : GenerationKind.None;
return true; return true;
} }
@ -65,8 +65,8 @@ namespace CppSharp.Passes
if (decl.IsDependent) if (decl.IsDependent)
{ {
decl.GenerationKind = decl is Field ? GenerationKind.Internal : GenerationKind.None; decl.GenerationKind = decl is Field ? GenerationKind.Internal : GenerationKind.None;
Diagnostics.Debug("'{0}' was ignored due to dependent context", Diagnostics.Debug("Decl '{0}' was ignored due to dependent context",
decl.QualifiedName); decl.Name);
return true; return true;
} }
@ -94,8 +94,8 @@ namespace CppSharp.Passes
var cppTypePrinter = new CppTypePrinter(); var cppTypePrinter = new CppTypePrinter();
var typeName = field.Type.Visit(cppTypePrinter); var typeName = field.Type.Visit(cppTypePrinter);
Diagnostics.Debug("'{0}' was ignored due to {1} type '{2}'", Diagnostics.Debug("Field '{0}::{1}' was ignored due to {2} type '{3}'",
field.QualifiedName, msg, typeName); @class.Name, field.Name, msg, typeName);
return true; return true;
} }
@ -111,8 +111,8 @@ namespace CppSharp.Passes
if (HasInvalidType(ret.Type, out msg)) if (HasInvalidType(ret.Type, out msg))
{ {
function.ExplicitlyIgnore(); function.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} return decl", Diagnostics.Debug("Function '{0}' was ignored due to {1} return decl",
function.QualifiedName, msg); function.Name, msg);
return false; return false;
} }
@ -121,16 +121,16 @@ namespace CppSharp.Passes
if (HasInvalidDecl(param, out msg)) if (HasInvalidDecl(param, out msg))
{ {
function.ExplicitlyIgnore(); function.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} param", Diagnostics.Debug("Function '{0}' was ignored due to {1} param",
function.QualifiedName, msg); function.Name, msg);
return false; return false;
} }
if (HasInvalidType(param.Type, out msg)) if (HasInvalidType(param.Type, out msg))
{ {
function.ExplicitlyIgnore(); function.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} param", Diagnostics.Debug("Function '{0}' was ignored due to {1} param",
function.QualifiedName, msg); function.Name, msg);
return false; return false;
} }
@ -138,8 +138,8 @@ namespace CppSharp.Passes
if (decayedType != null) if (decayedType != null)
{ {
function.ExplicitlyIgnore(); function.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to unsupported decayed type param", Diagnostics.Debug("Function '{0}' was ignored due to unsupported decayed type param",
function.QualifiedName); function.Name);
return false; return false;
} }
@ -151,8 +151,8 @@ namespace CppSharp.Passes
{ {
function.ExplicitlyIgnore(); function.ExplicitlyIgnore();
Diagnostics.Debug( Diagnostics.Debug(
"'{0}' was ignored due to an indirect return param not of a tag type", "Function '{0}' was ignored due to an indirect return param not of a tag type",
function.QualifiedName); function.Name);
return false; return false;
} }
} }
@ -179,8 +179,8 @@ namespace CppSharp.Passes
if (HasIgnoredBaseClass(method, @class, out ignoredBase)) if (HasIgnoredBaseClass(method, @class, out ignoredBase))
{ {
Diagnostics.Debug( Diagnostics.Debug(
"'{0}' was ignored due to ignored base '{1}'", "Virtual method '{0}' was ignored due to ignored base '{1}'",
method.QualifiedName, ignoredBase.Name); method.QualifiedOriginalName, ignoredBase.Name);
method.ExplicitlyIgnore(); method.ExplicitlyIgnore();
return false; return false;
@ -225,8 +225,8 @@ namespace CppSharp.Passes
if (HasInvalidType(typedef.Type, out msg)) if (HasInvalidType(typedef.Type, out msg))
{ {
typedef.ExplicitlyIgnore(); typedef.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} type", Diagnostics.Debug("Typedef '{0}' was ignored due to {1} type",
typedef.QualifiedName, msg); typedef.Name, msg);
return false; return false;
} }
@ -242,16 +242,16 @@ namespace CppSharp.Passes
if (HasInvalidDecl(property, out msg)) if (HasInvalidDecl(property, out msg))
{ {
property.ExplicitlyIgnore(); property.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} decl", Diagnostics.Debug("Property '{0}' was ignored due to {1} decl",
property.QualifiedName, msg); property.Name, msg);
return false; return false;
} }
if (HasInvalidType(property.Type, out msg)) if (HasInvalidType(property.Type, out msg))
{ {
property.ExplicitlyIgnore(); property.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} type", Diagnostics.Debug("Property '{0}' was ignored due to {1} type",
property.QualifiedName, msg); property.Name, msg);
return false; return false;
} }
@ -267,16 +267,16 @@ namespace CppSharp.Passes
if (HasInvalidDecl(variable, out msg)) if (HasInvalidDecl(variable, out msg))
{ {
variable.ExplicitlyIgnore(); variable.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} decl", Diagnostics.Debug("Variable '{0}' was ignored due to {1} decl",
variable.QualifiedName, msg); variable.Name, msg);
return false; return false;
} }
if (HasInvalidType(variable.Type, out msg)) if (HasInvalidType(variable.Type, out msg))
{ {
variable.ExplicitlyIgnore(); variable.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} type", Diagnostics.Debug("Variable '{0}' was ignored due to {1} type",
variable.QualifiedName, msg); variable.Name, msg);
return false; return false;
} }
@ -292,8 +292,8 @@ namespace CppSharp.Passes
if (HasInvalidDecl(@event, out msg)) if (HasInvalidDecl(@event, out msg))
{ {
@event.ExplicitlyIgnore(); @event.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} decl", Diagnostics.Debug("Event '{0}' was ignored due to {1} decl",
@event.QualifiedName, msg); @event.Name, msg);
return false; return false;
} }
@ -302,16 +302,16 @@ namespace CppSharp.Passes
if (HasInvalidDecl(param, out msg)) if (HasInvalidDecl(param, out msg))
{ {
@event.ExplicitlyIgnore(); @event.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} param", Diagnostics.Debug("Event '{0}' was ignored due to {1} param",
@event.QualifiedName, msg); @event.Name, msg);
return false; return false;
} }
if (HasInvalidType(param.Type, out msg)) if (HasInvalidType(param.Type, out msg))
{ {
@event.ExplicitlyIgnore(); @event.ExplicitlyIgnore();
Diagnostics.Debug("'{0}' was ignored due to {1} param", Diagnostics.Debug("Event '{0}' was ignored due to {1} param",
@event.QualifiedName, msg); @event.Name, msg);
return false; return false;
} }
} }

Loading…
Cancel
Save