Add PixelComponentPrecision

Author: JimBobSquarePants

src/ImageSharp/Color/Color.cs

@@ -107,7 +107,7 @@ private Color(IPixel pixel)
     public static Color FromPixel<TPixel>(TPixel pixel)
         where TPixel : unmanaged, IPixel<TPixel>
     {
-        // Avoid boxing in case we can convert to Rgba64 safely and efficently
+        // Avoid boxing in case we can convert to Rgba64 safely and efficiently
         if (typeof(TPixel) == typeof(Rgba64))
         {
             return new((Rgba64)(object)pixel);

src/ImageSharp/Formats/PixelTypeInfo.cs

@@ -4,45 +4,51 @@
 using System.Runtime.CompilerServices;
 using SixLabors.ImageSharp.PixelFormats;
 
-// TODO: Review this class as it's used to represent 2 different things.
+// TODO: Review this type as it's used to represent 2 different things.
 // 1.The encoded image pixel format.
 // 2. The pixel format of the decoded image.
 namespace SixLabors.ImageSharp.Formats;
 
 /// <summary>
 /// Contains information about the pixels that make up an images visual data.
 /// </summary>
-public readonly struct PixelTypeInfo
+/// <remarks>
+/// Initializes a new instance of the <see cref="PixelTypeInfo"/> struct.
+/// </remarks>
+/// <param name="bitsPerPixel">Color depth, in number of bits per pixel.</param>
+public readonly struct PixelTypeInfo(int bitsPerPixel)
 {
-    /// <summary>
-    /// Initializes a new instance of the <see cref="PixelTypeInfo"/> struct.
-    /// </summary>
-    /// <param name="bitsPerPixel">Color depth, in number of bits per pixel.</param>
-    public PixelTypeInfo(int bitsPerPixel)
-        => this.BitsPerPixel = bitsPerPixel;
-
     /// <summary>
     /// Gets color depth, in number of bits per pixel.
     /// </summary>
-    public int BitsPerPixel { get; init; }
+    public int BitsPerPixel { get; init; } = bitsPerPixel;
 
     /// <summary>
     /// Gets the count of the color components
     /// </summary>
     public byte ComponentCount { get; init; }
 
+    /// <summary>
+    /// Gets the pixel component precision.
+    /// </summary>
+    public PixelComponentPrecision? ComponentPrecision { get; init; }
+
     /// <summary>
     /// Gets the pixel alpha transparency behavior.
     /// <see langword="null"/> means unknown, unspecified.
     /// </summary>
     public PixelAlphaRepresentation? AlphaRepresentation { get; init; }
 
-    internal static PixelTypeInfo Create<TPixel>(byte componentCount, PixelAlphaRepresentation pixelAlphaRepresentation)
+    internal static PixelTypeInfo Create<TPixel>(
+        byte componentCount,
+        PixelComponentPrecision componentPrecision,
+        PixelAlphaRepresentation pixelAlphaRepresentation)
         where TPixel : unmanaged, IPixel<TPixel>
         => new()
         {
             BitsPerPixel = Unsafe.SizeOf<TPixel>() * 8,
             ComponentCount = componentCount,
+            ComponentPrecision = componentPrecision,
             AlphaRepresentation = pixelAlphaRepresentation
         };
 }

src/ImageSharp/Formats/Tiff/PhotometricInterpretation/CieLabPlanarTiffColor{TPixel}.cs

@@ -27,17 +27,17 @@ public override void Decode(IMemoryOwner<byte>[] data, Buffer2D<TPixel> pixels,
         Span<byte> a = data[1].GetSpan();
         Span<byte> b = data[2].GetSpan();
 
-        var color = default(TPixel);
+        TPixel color = default;
         int offset = 0;
         for (int y = top; y < top + height; y++)
         {
             Span<TPixel> pixelRow = pixels.DangerousGetRowSpan(y).Slice(left, width);
             for (int x = 0; x < pixelRow.Length; x++)
             {
-                var lab = new CieLab((l[offset] & 0xFF) * 100f * Inv255, (sbyte)a[offset], (sbyte)b[offset]);
-                var rgb = ColorSpaceConverter.ToRgb(lab);
+                CieLab lab = new((l[offset] & 0xFF) * 100f * Inv255, (sbyte)a[offset], (sbyte)b[offset]);
+                Rgb rgb = ColorSpaceConverter.ToRgb(lab);
 
-                color.FromVector4(new Vector4(rgb.R, rgb.G, rgb.B, 1.0f));
+                color.FromScaledVector4(new Vector4(rgb.R, rgb.G, rgb.B, 1.0f));
                 pixelRow[x] = color;
 
                 offset++;

src/ImageSharp/Formats/Tiff/PhotometricInterpretation/CieLabTiffColor{TPixel}.cs

@@ -34,7 +34,7 @@ public override void Decode(ReadOnlySpan<byte> data, Buffer2D<TPixel> pixels, in
                 CieLab lab = new(l, (sbyte)data[offset + 1], (sbyte)data[offset + 2]);
                 Rgb rgb = ColorSpaceConverter.ToRgb(lab);
 
-                color.FromVector4(new Vector4(rgb.R, rgb.G, rgb.B, 1.0f));
+                color.FromScaledVector4(new Vector4(rgb.R, rgb.G, rgb.B, 1.0f));
                 pixelRow[x] = color;
 
                 offset += 3;

src/ImageSharp/Formats/Tiff/PhotometricInterpretation/CmykTiffColor{TPixel}.cs

@@ -27,7 +27,7 @@ public override void Decode(ReadOnlySpan<byte> data, Buffer2D<TPixel> pixels, in
                 Cmyk cmyk = new(data[offset] * Inv255, data[offset + 1] * Inv255, data[offset + 2] * Inv255, data[offset + 3] * Inv255);
                 Rgb rgb = ColorSpaceConverter.ToRgb(in cmyk);
 
-                color.FromVector4(new Vector4(rgb.R, rgb.G, rgb.B, 1.0f));
+                color.FromScaledVector4(new Vector4(rgb.R, rgb.G, rgb.B, 1.0f));
                 pixelRow[x] = color;
 
                 offset += 4;

src/ImageSharp/PixelFormats/PixelComponentPrecision.cs

@@ -0,0 +1,50 @@
+// Copyright (c) Six Labors.
+// Licensed under the Six Labors Split License.
+
+namespace SixLabors.ImageSharp.PixelFormats;
+
+/// <summary>
+/// Provides enumeration of the maximum precision of individual components within a pixel format.
+/// </summary>
+public enum PixelComponentPrecision
+{
+    /// <summary>
+    /// 8-bit signed integer.
+    /// </summary>
+    SByte,
+
+    /// <summary>
+    /// 8-bit unsigned integer.
+    /// </summary>
+    Byte,
+
+    /// <summary>
+    /// 16-bit signed integer.
+    /// </summary>
+    Short,
+
+    /// <summary>
+    /// 16-bit unsigned integer.
+    /// </summary>
+    UShort,
+
+    /// <summary>
+    /// 32-bit signed integer.
+    /// </summary>
+    Int,
+
+    /// <summary>
+    /// 32-bit unsigned integer.
+    /// </summary>
+    UInt,
+
+    /// <summary>
+    /// 16-bit floating point.
+    /// </summary>
+    Half,
+
+    /// <summary>
+    /// 32-bit floating point.
+    /// </summary>
+    Float
+}

src/ImageSharp/PixelFormats/PixelImplementations/A8.cs

@@ -57,7 +57,7 @@ public partial struct A8 : IPixel<A8>, IPackedVector<byte>
     public static bool operator !=(A8 left, A8 right) => !left.Equals(right);
 
     /// <inheritdoc/>
-    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<A8>(1, PixelAlphaRepresentation.Unassociated);
+    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<A8>(1, PixelComponentPrecision.Byte, PixelAlphaRepresentation.Unassociated);
 
     /// <inheritdoc />
     public readonly PixelOperations<A8> CreatePixelOperations() => new PixelOperations();

src/ImageSharp/PixelFormats/PixelImplementations/Abgr32.cs

@@ -185,7 +185,7 @@ public uint PackedValue
     public static bool operator !=(Abgr32 left, Abgr32 right) => !left.Equals(right);
 
     /// <inheritdoc />
-    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<Abgr32>(4, PixelAlphaRepresentation.Unassociated);
+    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<Abgr32>(4, PixelComponentPrecision.Byte, PixelAlphaRepresentation.Unassociated);
 
     /// <inheritdoc />
     public readonly PixelOperations<Abgr32> CreatePixelOperations() => new PixelOperations();

src/ImageSharp/PixelFormats/PixelImplementations/Argb32.cs

@@ -185,7 +185,7 @@ public uint PackedValue
     public static bool operator !=(Argb32 left, Argb32 right) => !left.Equals(right);
 
     /// <inheritdoc />
-    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<Argb32>(4, PixelAlphaRepresentation.Unassociated);
+    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<Argb32>(4, PixelComponentPrecision.Byte, PixelAlphaRepresentation.Unassociated);
 
     /// <inheritdoc />
     public readonly PixelOperations<Argb32> CreatePixelOperations() => new PixelOperations();

src/ImageSharp/PixelFormats/PixelImplementations/Bgr24.cs

@@ -89,7 +89,7 @@ public Bgr24(byte r, byte g, byte b)
     public static bool operator !=(Bgr24 left, Bgr24 right) => !left.Equals(right);
 
     /// <inheritdoc />
-    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<Bgr24>(3, PixelAlphaRepresentation.None);
+    public static PixelTypeInfo GetPixelTypeInfo() => PixelTypeInfo.Create<Bgr24>(3, PixelComponentPrecision.Byte, PixelAlphaRepresentation.None);
 
     /// <inheritdoc/>
     public readonly PixelOperations<Bgr24> CreatePixelOperations() => new PixelOperations();