From 7c4d05c06e802aadbb392d3036ae873fbb0e453a Mon Sep 17 00:00:00 2001
From: Anton Firszov <anton.firszov@sapagroup.com>
Date: Sat, 18 Feb 2017 15:09:16 +0100
Subject: [PATCH] IPixel

---
 src/ImageSharp/Colors/PackedPixel/IPixel.cs   | 68 +++++++++++++++++++
 .../Common/Memory/ArrayPointer{T}.cs          |  4 +-
 2 files changed, 70 insertions(+), 2 deletions(-)
 create mode 100644 src/ImageSharp/Colors/PackedPixel/IPixel.cs

diff --git a/src/ImageSharp/Colors/PackedPixel/IPixel.cs b/src/ImageSharp/Colors/PackedPixel/IPixel.cs
new file mode 100644
index 000000000..a222edd9a
--- /dev/null
+++ b/src/ImageSharp/Colors/PackedPixel/IPixel.cs
@@ -0,0 +1,68 @@
+namespace ImageSharp
+{
+    using System;
+    using System.Numerics;
+
+    public interface IPixel
+    {
+        /// <summary>
+        /// Sets the packed representation from a <see cref="Vector4"/>.
+        /// </summary>
+        /// <param name="vector">The vector to create the packed representation from.</param>
+        void PackFromVector4(Vector4 vector);
+
+        /// <summary>
+        /// Expands the packed representation into a <see cref="Vector4"/>.
+        /// The vector components are typically expanded in least to greatest significance order.
+        /// </summary>
+        /// <returns>The <see cref="Vector4"/>.</returns>
+        Vector4 ToVector4();
+
+        /// <summary>
+        /// Sets the packed representation from the given byte array.
+        /// </summary>
+        /// <param name="x">The x-component.</param>
+        /// <param name="y">The y-component.</param>
+        /// <param name="z">The z-component.</param>
+        /// <param name="w">The w-component.</param>
+        void PackFromBytes(byte x, byte y, byte z, byte w);
+
+        /// <summary>
+        /// Expands the packed representation into a given byte array.
+        /// Output is expanded to X-> Y-> Z order. Equivalent to R-> G-> B in <see cref="Color"/>
+        /// </summary>
+        /// <param name="bytes">The bytes to set the color in.</param>
+        /// <param name="startIndex">The starting index of the <paramref name="bytes"/>.</param>
+        void ToXyzBytes(byte[] bytes, int startIndex);
+
+        /// <summary>
+        /// Expands the packed representation into a given byte array.
+        /// Output is expanded to X-> Y-> Z-> W order. Equivalent to R-> G-> B-> A in <see cref="Color"/>
+        /// </summary>
+        /// <param name="bytes">The bytes to set the color in.</param>
+        /// <param name="startIndex">The starting index of the <paramref name="bytes"/>.</param>
+        void ToXyzwBytes(byte[] bytes, int startIndex);
+
+        /// <summary>
+        /// Expands the packed representation into a given byte array.
+        /// Output is expanded to Z-> Y-> X order. Equivalent to B-> G-> R in <see cref="Color"/>
+        /// </summary>
+        /// <param name="bytes">The bytes to set the color in.</param>
+        /// <param name="startIndex">The starting index of the <paramref name="bytes"/>.</param>
+        void ToZyxBytes(byte[] bytes, int startIndex);
+
+        /// <summary>
+        /// Expands the packed representation into a given byte array.
+        /// Output is expanded to Z-> Y-> X-> W order. Equivalent to B-> G-> R-> A in <see cref="Color"/>
+        /// </summary>
+        /// <param name="bytes">The bytes to set the color in.</param>
+        /// <param name="startIndex">The starting index of the <paramref name="bytes"/>.</param>
+        void ToZyxwBytes(byte[] bytes, int startIndex);
+    }
+
+    public interface IPixel<TSelf> : IPixel, IEquatable<TSelf>
+        where TSelf : struct, IPixel<TSelf>
+    {
+        
+    }
+}
\ No newline at end of file
diff --git a/src/ImageSharp/Common/Memory/ArrayPointer{T}.cs b/src/ImageSharp/Common/Memory/ArrayPointer{T}.cs
index c3fc32234..06de06107 100644
--- a/src/ImageSharp/Common/Memory/ArrayPointer{T}.cs
+++ b/src/ImageSharp/Common/Memory/ArrayPointer{T}.cs
@@ -16,7 +16,7 @@ namespace ImageSharp
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public ArrayPointer(T[] array, void* pointerToArray, int offset)
         {
-            // TODO: Use Guard.NotNull() here after optimizing it with ThrowHelper!
+            // TODO: Use Guard.NotNull() here after optimizing it by eliminating the default argument case and applying ThrowHelper!
             if (array == null)
             {
                 ThrowHelper.ThrowArgumentNullException(nameof(array));
@@ -29,7 +29,7 @@ namespace ImageSharp
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public ArrayPointer(T[] array, void* pointerToArray)
         {
-            // TODO: Use Guard.NotNull() here after optimizing it with ThrowHelper!
+            // TODO: Use Guard.NotNull() here after optimizing it by eliminating the default argument case and applying ThrowHelper!
             if (array == null)
             {
                 ThrowHelper.ThrowArgumentNullException(nameof(array));