Use Asserts instead of null checks because this class is internal.

Author: Dirk Lemstra

Source/Magick.NET/Core/Drawables/DrawingWand.cs

@@ -14,6 +14,8 @@
 
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
+using System.Diagnostics.CodeAnalysis;
 using System.Text;
 
 namespace ImageMagick
@@ -100,8 +102,8 @@ public void Density(PointD value)
 
     public void Dispose()
     {
-      if (_NativeInstance != null)
-        _NativeInstance.Dispose();
+      Debug.Assert(_NativeInstance != null);
+      _NativeInstance.Dispose();
     }
 
     public void Ellipse(double originX, double originY, double radiusX, double radiusY, double startDegrees, double endDegrees)
@@ -154,21 +156,21 @@ public void Line(double startX, double startY, double endX, double endY)
       _NativeInstance.Line(startX, startY, endX, endY);
     }
 
+    [SuppressMessage("Microsoft.Design", "CA1062:Validate arguments of public methods", MessageId = "0", Justification = "Using assert instead.")]
     public void PathArcAbs(IEnumerable<PathArc> pathArcs)
     {
-      if (pathArcs == null)
-        return;
+      Debug.Assert(pathArcs != null);
 
       foreach (PathArc pathArc in pathArcs)
       {
         _NativeInstance.PathArcAbs(pathArc.X, pathArc.Y, pathArc.RadiusX, pathArc.RadiusY, pathArc.RotationX, pathArc.UseLargeArc, pathArc.UseSweep);
       }
     }
 
+    [SuppressMessage("Microsoft.Design", "CA1062:Validate arguments of public methods", MessageId = "0", Justification = "Using assert instead.")]
     public void PathArcRel(IEnumerable<PathArc> pathArcs)
     {
-      if (pathArcs == null)
-        return;
+      Debug.Assert(pathArcs != null);
 
       foreach (PathArc pathArc in pathArcs)
       {
@@ -196,10 +198,10 @@ public void PathFinish()
       _NativeInstance.PathFinish();
     }
 
+    [SuppressMessage("Microsoft.Design", "CA1062:Validate arguments of public methods", MessageId = "0", Justification = "Using assert instead.")]
     public void PathLineToAbs(IEnumerable<PointD> coordinates)
     {
-      if (coordinates == null)
-        return;
+      Debug.Assert(coordinates != null);
 
       foreach (PointD coordinate in coordinates)
       {
@@ -227,10 +229,10 @@ public void PathLineToVerticalAbs(double y)
       _NativeInstance.PathLineToVerticalAbs(y);
     }
 
+    [SuppressMessage("Microsoft.Design", "CA1062:Validate arguments of public methods", MessageId = "0", Justification = "Using assert instead.")]
     public void PathLineToRel(IEnumerable<PointD> coordinates)
     {
-      if (coordinates == null)
-        return;
+      Debug.Assert(coordinates != null);
 
       foreach (PointD coordinate in coordinates)
       {
@@ -290,8 +292,7 @@ public void Point(double x, double y)
 
     public void Polygon(IList<PointD> coordinates)
     {
-      if (coordinates == null)
-        return;
+      Debug.Assert(coordinates != null);
 
       using (PointInfoCollection pointInfo = new PointInfoCollection(coordinates))
       {
@@ -301,8 +302,7 @@ public void Polygon(IList<PointD> coordinates)
 
     public void Polyline(IList<PointD> coordinates)
     {
-      if (coordinates == null)
-        return;
+      Debug.Assert(coordinates != null);
 
       using (PointInfoCollection pointInfo = new PointInfoCollection(coordinates))
       {
@@ -380,10 +380,12 @@ public void StrokeColor(MagickColor color)
       _NativeInstance.StrokeColor(color);
     }
 
+    [SuppressMessage("Microsoft.Design", "CA1062:Validate arguments of public methods", MessageId = "0", Justification = "Using assert instead.")]
     public void StrokeDashArray(double[] dash)
     {
-      if (dash != null)
-        _NativeInstance.StrokeDashArray(dash, dash.Length);
+      Debug.Assert(dash != null);
+
+      _NativeInstance.StrokeDashArray(dash, dash.Length);
     }
 
     public void StrokeDashOffset(double value)