Read data for PropertyIdentifierAndOffset using the Read() member fun…

…ction Just a thought - the code coverage report shows the Read() function as unused (the Write function is used for writes, so this is maybe a bit more consistent)
ironfede · Nov 19, 2024 · 0e8ecab · 0e8ecab
1 parent 1a9b982
commit 0e8ecab
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/OpenMcdf.Ole/PropertySetStream.cs b/OpenMcdf.Ole/PropertySetStream.cs
@@ -58,11 +58,8 @@ public void Read(BinaryReader br)
         // Read property offsets (P0)
         for (int i = 0; i < propertyCount; i++)
         {
-            PropertyIdentifierAndOffset pio = new()
-            {
-                PropertyIdentifier = br.ReadUInt32(),
-                Offset = br.ReadUInt32()
-            };
+            PropertyIdentifierAndOffset pio = new();
+            pio.Read(br);
             PropertySet0.PropertyIdentifierAndOffsets.Add(pio);
         }
 
@@ -92,11 +89,8 @@ public void Read(BinaryReader br)
             // Read property offsets
             for (int i = 0; i < propertyCount; i++)
             {
-                PropertyIdentifierAndOffset pio = new()
-                {
-                    PropertyIdentifier = br.ReadUInt32(),
-                    Offset = br.ReadUInt32()
-                };
+                PropertyIdentifierAndOffset pio = new();
+                pio.Read(br);
                 PropertySet1.PropertyIdentifierAndOffsets.Add(pio);
             }