Use ArgumentNullException.ThrowIfNull() to check arguments

Author: jozefizso

src/Kavod.Vba.Compression/CompressedChunk.cs

@@ -16,10 +16,7 @@ internal class CompressedChunk
     {
         internal CompressedChunk(DecompressedChunk decompressedChunk)
         {
-            if (decompressedChunk == null)
-            {
-                throw new ArgumentNullException(nameof(decompressedChunk));
-            }
+            ArgumentNullException.ThrowIfNull(decompressedChunk);
 
             ChunkData = new CompressedChunkData(decompressedChunk);
             if (ChunkData.Size >= Globals.MaxBytesPerChunk)
@@ -31,10 +28,7 @@ internal CompressedChunk(DecompressedChunk decompressedChunk)
 
         internal CompressedChunk(BinaryReader dataReader)
         {
-            if (dataReader == null)
-            {
-                throw new ArgumentNullException(nameof(dataReader));
-            }
+            ArgumentNullException.ThrowIfNull(dataReader);
 
             Header = new CompressedChunkHeader(dataReader);
             if (Header.IsCompressed)

src/Kavod.Vba.Compression/CompressedChunkData.cs

@@ -18,10 +18,7 @@ internal class CompressedChunkData : IChunkData
 
         internal CompressedChunkData(DecompressedChunk chunk)
         {
-            if (chunk == null)
-            {
-                throw new ArgumentNullException(nameof(chunk));
-            }
+            ArgumentNullException.ThrowIfNull(chunk);
 
             var tokens = Tokenizer.TokenizeUncompressedData(chunk.Data);
             _tokensequences.AddRange(tokens.ToTokenSequences());

src/Kavod.Vba.Compression/Tokenizer.cs

@@ -162,10 +162,7 @@ private static IEnumerable<CopyToken> RemoveOverlappingTokens(IEnumerable<CopyTo
 
         private static Node FindBestPath(Node node)
         {
-            if (node == null)
-            {
-                throw new ArgumentNullException(nameof(node));
-            }
+            ArgumentNullException.ThrowIfNull(node);
 
             // find any overlapping tokens
             Node bestPath = null;
@@ -191,10 +188,7 @@ private static Node FindBestPath(Node node)
 
         private static IEnumerable<Node> GetOverlappingNodes(Node node)
         {
-            if (node == null)
-            {
-                throw new ArgumentNullException(nameof(node));
-            }
+            ArgumentNullException.ThrowIfNull(node);
 
             var firstNode = node;
 
@@ -208,10 +202,7 @@ private static IEnumerable<Node> GetOverlappingNodes(Node node)
 
         private static Node GetNextNonOverlappingNode(Node node)
         {
-            if (node == null)
-            {
-                throw new ArgumentNullException(nameof(node));
-            }
+            ArgumentNullException.ThrowIfNull(node);
 
             var firstNode = node;
 
@@ -322,10 +313,7 @@ private class Node : IEnumerable<CopyToken>
         {
             public Node(CopyToken value, Node nextNode)
             {
-                if (value == null)
-                {
-                    throw new ArgumentNullException(nameof(value));
-                }
+                ArgumentNullException.ThrowIfNull(value);
 
                 Value = value;
                 NextNode = nextNode;