Merge branch 'ExcelDataReader:develop' into fix-578-issue

Author: Andreybonfi

src/ExcelDataReader/Core/BuiltinNumberFormat.cs

@@ -5,7 +5,7 @@ namespace ExcelDataReader.Core
 {
     internal static class BuiltinNumberFormat
     {
-        private static Dictionary<int, NumberFormatString> Formats { get; } = new Dictionary<int, NumberFormatString>()
+        private static Dictionary<int, NumberFormatString> Formats { get; } = new Dictionary<int, NumberFormatString>
         {
             { 0, new NumberFormatString("General") },
             { 1, new NumberFormatString("0") },

src/ExcelDataReader/Core/Helpers.cs

@@ -1,4 +1,3 @@
-using System;
 using System.Globalization;
 using System.Text;
 using System.Text.RegularExpressions;
@@ -11,7 +10,7 @@ namespace ExcelDataReader.Core
     /// </summary>
     internal static class Helpers
     {
-        private static readonly Regex EscapeRegex = new("_x([0-9A-F]{4,4})_");
+        private static readonly Regex EscapeRegex = new("_x([0-9A-F]{4,4})_", RegexOptions.Compiled);
 
         /// <summary>
         /// Determines whether the encoding is single byte or not.

src/ExcelDataReader/Core/OpenXmlFormat/BinaryFormat/BiffReader.cs

@@ -27,7 +27,7 @@ public BiffReader(Stream stream)
                 return null;
 
             byte[] buffer = recordLength < _buffer.Length ? _buffer : new byte[recordLength];
-            if (Stream.Read(buffer, 0, (int)recordLength) != recordLength)
+            if (Stream.ReadAtLeast(buffer, 0, (int)recordLength) != recordLength)
                 return null;
 
             return ReadOverride(buffer, recordId, recordLength);

src/ExcelDataReader/Core/OpenXmlFormat/Records/Record.cs

@@ -1,8 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Text;
-
-namespace ExcelDataReader.Core.OpenXmlFormat.Records
+namespace ExcelDataReader.Core.OpenXmlFormat.Records
 {
     internal abstract class Record
     {

src/ExcelDataReader/Core/OpenXmlFormat/ZipWorker.cs

@@ -64,27 +64,27 @@ public ZipWorker(Stream fileStream)
             using var reader = XmlReader.Create(workbookRelsEntry.Open(), XmlSettings);
             while (reader.Read())
             {
-                if (reader.NodeType == XmlNodeType.Element && reader.Name == "Relationship")
-                {
-                    var id = reader.GetAttribute("Id");
-                    var type = reader.GetAttribute("Type");
-                    var target = reader.GetAttribute("Target");
+                if (reader.NodeType != XmlNodeType.Element || reader.Name != "Relationship")
+                    continue;
 
-                    switch (type)
-                    {
-                        case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet":
-                        case "http://purl.oclc.org/ooxml/officeDocument/relationships/worksheet":
-                            _worksheetRels[id] = ResolvePath(basePath, target);
-                            break;
-                        case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/styles":
-                        case "http://purl.oclc.org/ooxml/officeDocument/relationships/styles":
-                            _fileStyles = ResolvePath(basePath, target);
-                            break;
-                        case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/sharedStrings":
-                        case "http://purl.oclc.org/ooxml/officeDocument/relationships/sharedStrings":
-                            _fileSharedStrings = ResolvePath(basePath, target);
-                            break;
-                    }
+                var id = reader.GetAttribute("Id");
+                var type = reader.GetAttribute("Type");
+                var target = reader.GetAttribute("Target");
+
+                switch (type)
+                {
+                    case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet":
+                    case "http://purl.oclc.org/ooxml/officeDocument/relationships/worksheet":
+                        _worksheetRels[id] = ResolvePath(basePath, target);
+                        break;
+                    case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/styles":
+                    case "http://purl.oclc.org/ooxml/officeDocument/relationships/styles":
+                        _fileStyles = ResolvePath(basePath, target);
+                        break;
+                    case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/sharedStrings":
+                    case "http://purl.oclc.org/ooxml/officeDocument/relationships/sharedStrings":
+                        _fileSharedStrings = ResolvePath(basePath, target);
+                        break;
                 }
             }
 
@@ -112,17 +112,17 @@ static string ResolvePath(string? basePath, string path)
                 using var reader = XmlReader.Create(entry.Open(), XmlSettings);
                 while (reader.Read())
                 {
-                    if (reader.NodeType == XmlNodeType.Element && reader.Name == "Relationship")
+                    if (reader.NodeType != XmlNodeType.Element || reader.Name != "Relationship")
+                        continue;
+
+                    var type = reader.GetAttribute("Type");
+                    var target = reader.GetAttribute("Target");
+
+                    switch (type)
                     {
-                        var type = reader.GetAttribute("Type");
-                        var target = reader.GetAttribute("Target");
-
-                        switch (type)
-                        {
-                            case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument":
-                            case "http://purl.oclc.org/ooxml/officeDocument/relationships/officeDocument":
-                                return target;
-                        }
+                        case "http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument":
+                        case "http://purl.oclc.org/ooxml/officeDocument/relationships/officeDocument":
+                            return target;
                     }
                 }
 

src/ExcelDataReader/Misc/DateTimeHelper.cs

@@ -55,7 +55,7 @@ public static DateTime FromOADate(double d)
         internal static long DoubleDateToTicks(double value)
         {
             if (value >= OADateMaxAsDouble || value <= OADateMinAsDouble)
-                throw new ArgumentException("Invalid OA Date");
+                throw new ArgumentException("Invalid OA Date", nameof(value));
             long millis = (long)(value * MillisPerDay + (value >= 0 ? 0.5 : -0.5));
 
             // The interesting thing here is when you have a value like 12.5 it all positive 12 days and 12 hours from 01/01/1899