Updated null checking and variable declaration for modern language:

IDE0016  C# Null check can be simplified
IDE0018  C# Variable declaration can be inlined
IDE0044 ... well, yes, why not for this place :)
+CA1063 another way

Conari/Conari.csproj

@@ -20,6 +20,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <NoWarn>IDE1006</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <DebugType>pdbonly</DebugType>
@@ -30,7 +31,7 @@
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
     <DocumentationFile>bin\Release\Conari.XML</DocumentationFile>
-    <NoWarn>CS1591</NoWarn>
+    <NoWarn>CS1591,IDE1006</NoWarn>
   </PropertyGroup>
   <PropertyGroup>
     <SignAssembly>true</SignAssembly>

Conari/ConariL.cs

@@ -30,7 +30,6 @@
 
 namespace net.r_eg.Conari
 {
-    [SuppressMessage("Microsoft.Design", "CA1063:ImplementIDisposableCorrectly", Justification = "Bug. False positive. The IDisposable is already implemented correctly !")]
     public class ConariL: Provider, IConari, ILoader, IProvider, IBinder, IDisposable
     {
         protected IConfig config;
@@ -171,11 +170,7 @@ public ConariL(string lib, string prefix = null)
 
         protected void init(IConfig cfg)
         {
-            if(cfg == null) {
-                throw new ArgumentNullException("Configuration cannot be null.");
-            }
-
-            config      = cfg;
+            config      = cfg ?? throw new ArgumentNullException("Configuration cannot be null.");
             Mangling    = cfg.Mangling;
 
             if(cfg.LazyLoading) {
@@ -200,5 +195,15 @@ private void onConventionChanged(object sender, DataArgs<CallingConvention> e)
             DLR = newDLR(e.Data);
             LSender.Send(sender, $"DLR has been updated with new CallingConvention: {e.Data}", Message.Level.Info);
         }
+
+        #region IDisposable
+
+        // CA1063
+        protected override void Dispose(bool disposing)
+        {
+            base.Dispose(disposing);
+        }
+
+        #endregion
     }
 }

Conari/Core/ExVar.cs

@@ -198,10 +198,7 @@ public override IEnumerable<string> GetDynamicMemberNames()
 
         public ExVar(IProvider p)
         {
-            if(p == null) {
-                throw new ArgumentException("Provider cannot be null.");
-            }
-            provider = p;
+            provider = p ?? throw new ArgumentException("Provider cannot be null.");
         }
 
         protected dynamic getFieldDLR(Type type, string variable)

Conari/Extensions/ObjectExtension.cs

@@ -66,7 +66,7 @@ public static object Invoke(this Object obj, string name, BindingFlags flags, pa
 
                 return new Act(
                     mi != null,
-                    mi?.Invoke(obj, (args == null) ? new object[0] : args)
+                    mi?.Invoke(obj, args ?? new object[0])
                 );
             },
             obj, name);

Conari/Native/Core/BReader.cs

@@ -142,8 +142,7 @@ public bool tryNext(Type type, int tsize, out dynamic value)
         /// <exception cref="IndexOutOfRangeException"></exception>
         public dynamic next(Type type, int tsize)
         {
-            dynamic value;
-            if(tryNext(type, tsize, out value)) {
+            if(tryNext(type, tsize, out dynamic value)) {
                 return value;
             }
 

Conari/Native/NativeData.cs

@@ -36,7 +36,7 @@ public class NativeData
     {
         protected Fields map = new Fields();
         protected byte[] local;
-        private IntPtr pointer;
+        private readonly IntPtr pointer;
 
         /// <summary>
         /// Get raw-data of complex native structure.

ConariTest/ConariTest.csproj

@@ -26,6 +26,7 @@
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
     <PlatformTarget>x86</PlatformTarget>
+    <NoWarn>IDE1006</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <DebugType>pdbonly</DebugType>