From df3248ddf0a3d43fb9910fe00ec03e647181639f Mon Sep 17 00:00:00 2001
From: Jason Nelson <jason@carbonmade.com>
Date: Thu, 19 Apr 2018 13:00:06 -0700
Subject: [PATCH] Make IccProfileDescription a struct

---
 .../DataWriter/IccDataWriter.TagDataEntry.cs  |  5 +-
 .../ICC/Various/IccProfileDescription.cs      | 50 ++++++-------------
 2 files changed, 18 insertions(+), 37 deletions(-)

diff --git a/src/ImageSharp/MetaData/Profiles/ICC/DataWriter/IccDataWriter.TagDataEntry.cs b/src/ImageSharp/MetaData/Profiles/ICC/DataWriter/IccDataWriter.TagDataEntry.cs
index 210f0b5e5..18280faaf 100644
--- a/src/ImageSharp/MetaData/Profiles/ICC/DataWriter/IccDataWriter.TagDataEntry.cs
+++ b/src/ImageSharp/MetaData/Profiles/ICC/DataWriter/IccDataWriter.TagDataEntry.cs
@@ -686,8 +686,11 @@ namespace SixLabors.ImageSharp.MetaData.Profiles.Icc
         public int WriteProfileSequenceDescTagDataEntry(IccProfileSequenceDescTagDataEntry value)
         {
             int count = this.WriteUInt32((uint)value.Descriptions.Length);
-            foreach (IccProfileDescription desc in value.Descriptions)
+
+            for (int i = 0; i < value.Descriptions.Length; i++)
             {
+                ref IccProfileDescription desc = ref value.Descriptions[i];
+
                 count += this.WriteProfileDescription(desc);
             }
 
diff --git a/src/ImageSharp/MetaData/Profiles/ICC/Various/IccProfileDescription.cs b/src/ImageSharp/MetaData/Profiles/ICC/Various/IccProfileDescription.cs
index 32e2a951e..685319fe4 100644
--- a/src/ImageSharp/MetaData/Profiles/ICC/Various/IccProfileDescription.cs
+++ b/src/ImageSharp/MetaData/Profiles/ICC/Various/IccProfileDescription.cs
@@ -9,7 +9,7 @@ namespace SixLabors.ImageSharp.MetaData.Profiles.Icc
     /// <summary>
     /// ICC Profile description
     /// </summary>
-    internal sealed class IccProfileDescription : IEquatable<IccProfileDescription>
+    internal readonly struct IccProfileDescription : IEquatable<IccProfileDescription>
     {
         /// <summary>
         /// Initializes a new instance of the <see cref="IccProfileDescription"/> class.
@@ -40,69 +40,47 @@ namespace SixLabors.ImageSharp.MetaData.Profiles.Icc
         }
 
         /// <summary>
-        /// Gets the device manufacturer
+        /// Gets the device manufacturer.
         /// </summary>
         public uint DeviceManufacturer { get; }
 
         /// <summary>
-        /// Gets the device model
+        /// Gets the device model.
         /// </summary>
         public uint DeviceModel { get; }
 
         /// <summary>
-        /// Gets the device attributes
+        /// Gets the device attributes.
         /// </summary>
         public IccDeviceAttribute DeviceAttributes { get; }
 
         /// <summary>
-        /// Gets the technology information
+        /// Gets the technology information.
         /// </summary>
         public IccProfileTag TechnologyInformation { get; }
 
         /// <summary>
-        /// Gets the device manufacturer info
+        /// Gets the device manufacturer info.
         /// </summary>
         public IccLocalizedString[] DeviceManufacturerInfo { get; }
 
         /// <summary>
-        /// Gets the device model info
+        /// Gets the device model info.
         /// </summary>
         public IccLocalizedString[] DeviceModelInfo { get; }
 
         /// <inheritdoc/>
-        public bool Equals(IccProfileDescription other)
-        {
-            if (other == null)
-            {
-                return false;
-            }
-
-            if (ReferenceEquals(this, other))
-            {
-                return true;
-            }
-
-            return this.DeviceManufacturer == other.DeviceManufacturer
-                && this.DeviceModel == other.DeviceModel
-                && this.DeviceAttributes == other.DeviceAttributes
-                && this.TechnologyInformation == other.TechnologyInformation
-                && this.DeviceManufacturerInfo.SequenceEqual(other.DeviceManufacturerInfo)
-                && this.DeviceModelInfo.SequenceEqual(other.DeviceModelInfo);
-        }
+        public bool Equals(IccProfileDescription other) =>
+            this.DeviceManufacturer == other.DeviceManufacturer && 
+            this.DeviceModel == other.DeviceModel &&
+            this.DeviceAttributes == other.DeviceAttributes &&
+            this.TechnologyInformation == other.TechnologyInformation &&
+            this.DeviceManufacturerInfo.SequenceEqual(other.DeviceManufacturerInfo) &&
+            this.DeviceModelInfo.SequenceEqual(other.DeviceModelInfo);
 
         /// <inheritdoc />
         public override bool Equals(object obj)
         {
-            if (obj == null)
-            {
-                return false;
-            }
-
-            if (ReferenceEquals(this, obj))
-            {
-                return true;
-            }
-
             return obj is IccProfileDescription other && this.Equals(other);
         }