You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4net-dev@logging.apache.org by ni...@apache.org on 2004/12/19 20:23:42 UTC
cvs commit: logging-log4net/src/ObjectRenderer DefaultRenderer.cs
nicko 2004/12/19 11:23:42
Modified: src/ObjectRenderer DefaultRenderer.cs
Log:
Updates suggested by FxCop.
Minor naming changes to internal types.
Perf improvements to type check + type cast code.
Revision Changes Path
1.8 +17 -10 logging-log4net/src/ObjectRenderer/DefaultRenderer.cs
Index: DefaultRenderer.cs
===================================================================
RCS file: /home/cvs/logging-log4net/src/ObjectRenderer/DefaultRenderer.cs,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- DefaultRenderer.cs 6 Dec 2004 02:26:04 -0000 1.7
+++ DefaultRenderer.cs 19 Dec 2004 19:23:42 -0000 1.8
@@ -145,24 +145,31 @@
if (obj == null)
{
writer.Write("(null)");
+ return;
}
- else if (obj is Array)
+
+ Array objArray = obj as Array;
+ if (objArray != null)
{
- RenderArray(rendererMap, (Array)obj, writer);
+ RenderArray(rendererMap, objArray, writer);
+ return;
}
- else if (obj is ICollection)
+
+ ICollection objCollection = obj as ICollection;
+ if (objCollection != null)
{
- RenderCollection(rendererMap, (ICollection)obj, writer);
+ RenderCollection(rendererMap, objCollection, writer);
+ return;
}
- else if (obj is DictionaryEntry)
+
+ if (obj is DictionaryEntry)
{
RenderDictionaryEntry(rendererMap, (DictionaryEntry)obj, writer);
+ return;
}
- else
- {
- string str = obj.ToString();
- writer.Write( (str==null) ? "(null)" : str );
- }
+
+ string str = obj.ToString();
+ writer.Write( (str==null) ? "(null)" : str );
}
#endregion