From 02d83c9bd240062d67e94ee3b4d99971d22c2274 Mon Sep 17 00:00:00 2001
From: Ildar Khayrutdinov <ildarkhairutdin@gmail.com>
Date: Sat, 2 Oct 2021 20:38:53 +0300
Subject: [PATCH] minor

---
 .../Metadata/Profiles/Exif/ExifReader.cs      | 30 ++++++++++---------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/src/ImageSharp/Metadata/Profiles/Exif/ExifReader.cs b/src/ImageSharp/Metadata/Profiles/Exif/ExifReader.cs
index d9fe29e821..6107bd766c 100644
--- a/src/ImageSharp/Metadata/Profiles/Exif/ExifReader.cs
+++ b/src/ImageSharp/Metadata/Profiles/Exif/ExifReader.cs
@@ -114,6 +114,11 @@ namespace SixLabors.ImageSharp.Metadata.Profiles.Exif
 
         protected void ReadBigValues(List<IExifValue> values)
         {
+            if (this.BigValues.Count == 0)
+            {
+                return;
+            }
+
             ulong maxSize = 0;
             foreach ((ulong offset, ExifDataType dataType, ulong numberOfComponents, ExifValue exif) in this.BigValues)
             {
@@ -159,14 +164,12 @@ namespace SixLabors.ImageSharp.Metadata.Profiles.Exif
 
         protected void ReadSubIfd(List<IExifValue> values)
         {
-            if (this.subIfds is null)
-            {
-                return;
-            }
-
-            foreach (ulong subIfdOffset in this.subIfds)
+            if (this.subIfds is not null)
             {
-                this.ReadValues(values, (uint)subIfdOffset);
+                foreach (ulong subIfdOffset in this.subIfds)
+                {
+                    this.ReadValues(values, (uint)subIfdOffset);
+                }
             }
         }
 
@@ -196,14 +199,12 @@ namespace SixLabors.ImageSharp.Metadata.Profiles.Exif
 
         protected void ReadSubIfd64(List<IExifValue> values)
         {
-            if (this.subIfds is null)
-            {
-                return;
-            }
-
-            foreach (ulong subIfdOffset in this.subIfds)
+            if (this.subIfds is not null)
             {
-                this.ReadValues64(values, subIfdOffset);
+                foreach (ulong subIfdOffset in this.subIfds)
+                {
+                    this.ReadValues64(values, subIfdOffset);
+                }
             }
         }
 
@@ -517,6 +518,7 @@ namespace SixLabors.ImageSharp.Metadata.Profiles.Exif
             }
             else if (exif.Tag == ExifTag.SubIFDs)
             {
+                //// didn't find any useful data in SubIFDs
                 ////foreach (object val in (Array)value)
                 ////{
                 ////    this.AddSubIfd(val);