Revert "Fixed color component order (should be reversed) for Argb32, Rgba32, Bgr24, Rgb24, Bgra32. Tests still need to be updated."

This reverts commit a481a3e621.

src/ImageSharp/PixelFormats/Argb32.cs

@@ -22,24 +22,24 @@ namespace SixLabors.ImageSharp.PixelFormats
     public struct Argb32 : IPixel<Argb32>, IPackedVector<uint>
     {
         /// <summary>
-        /// Gets or sets the blue component.
+        /// Gets or sets the alpha component.
         /// </summary>
-        public byte B;
+        public byte A;
 
         /// <summary>
-        /// Gets or sets the green component.
+        /// Gets or sets the red component.
         /// </summary>
-        public byte G;
+        public byte R;
 
         /// <summary>
-        /// Gets or sets the red component.
+        /// Gets or sets the green component.
         /// </summary>
-        public byte R;
+        public byte G;
 
         /// <summary>
-        /// Gets or sets the alpha component.
+        /// Gets or sets the blue component.
         /// </summary>
-        public byte A;
+        public byte B;
 
         /// <summary>
         /// The maximum byte value.
@@ -153,24 +153,6 @@ namespace SixLabors.ImageSharp.PixelFormats
             }
         }
 
-        /// <summary>
-        /// Gets or sets the RGB components of this struct as <see cref="Rgb24"/>
-        /// </summary>
-        public Rgb24 Rgb
-        {
-            [MethodImpl(MethodImplOptions.AggressiveInlining)]
-            get
-            {
-                return Unsafe.As<Argb32, Rgb24>(ref this);
-            }
-
-            [MethodImpl(MethodImplOptions.AggressiveInlining)]
-            set
-            {
-                Unsafe.As<Argb32, Rgb24>(ref this) = value;
-            }
-        }
-
         /// <inheritdoc/>
         public uint PackedValue
         {

src/ImageSharp/PixelFormats/Bgr24.cs

@@ -18,9 +18,9 @@ namespace SixLabors.ImageSharp.PixelFormats
     public struct Bgr24 : IPixel<Bgr24>
     {
         /// <summary>
-        /// The red component.
+        /// The blue component.
         /// </summary>
-        public byte R;
+        public byte B;
 
         /// <summary>
         /// The green component.
@@ -28,9 +28,9 @@ namespace SixLabors.ImageSharp.PixelFormats
         public byte G;
 
         /// <summary>
-        /// The blue component.
+        /// The red component.
         /// </summary>
-        public byte B;
+        public byte R;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="Bgr24"/> struct.

src/ImageSharp/PixelFormats/Bgra32.cs

@@ -19,24 +19,24 @@ namespace SixLabors.ImageSharp.PixelFormats
     public struct Bgra32 : IPixel<Bgra32>, IPackedVector<uint>
     {
         /// <summary>
-        /// Gets or sets the alpha component.
+        /// Gets or sets the blue component.
         /// </summary>
-        public byte A;
+        public byte B;
 
         /// <summary>
-        /// Gets or sets the red component.
+        /// Gets or sets the green component.
         /// </summary>
-        public byte R;
+        public byte G;
 
         /// <summary>
-        /// Gets or sets the green component.
+        /// Gets or sets the red component.
         /// </summary>
-        public byte G;
+        public byte R;
 
         /// <summary>
-        /// Gets or sets the blue component.
+        /// Gets or sets the alpha component.
         /// </summary>
-        public byte B;
+        public byte A;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="Bgra32"/> struct.

src/ImageSharp/PixelFormats/Rgb24.cs

@@ -19,9 +19,9 @@ namespace SixLabors.ImageSharp.PixelFormats
     public struct Rgb24 : IPixel<Rgb24>
     {
         /// <summary>
-        /// The blue component.
+        /// The red component.
         /// </summary>
-        public byte B;
+        public byte R;
 
         /// <summary>
         /// The green component.
@@ -29,9 +29,9 @@ namespace SixLabors.ImageSharp.PixelFormats
         public byte G;
 
         /// <summary>
-        /// The red component.
+        /// The blue component.
         /// </summary>
-        public byte R;
+        public byte B;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="Rgb24"/> struct.
@@ -80,16 +80,16 @@ namespace SixLabors.ImageSharp.PixelFormats
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public void PackFromRgba32(Rgba32 source)
         {
-            this.R = source.R;
+            this = Unsafe.As<Rgba32, Rgb24>(ref source);
-            this.G = source.G;
-            this.B = source.B;
         }
 
         /// <inheritdoc/>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public void PackFromArgb32(Argb32 source)
         {
-            this = Unsafe.As<Argb32, Rgb24>(ref source);
+            this.R = source.R;
+            this.G = source.G;
+            this.B = source.B;
         }
 
         /// <inheritdoc/>
@@ -132,15 +132,15 @@ namespace SixLabors.ImageSharp.PixelFormats
         /// <inheritdoc/>
         public void ToRgba32(ref Rgba32 dest)
         {
-            dest.R = this.R;
+            dest.Rgb = this;
-            dest.G = this.G;
-            dest.B = this.B;
             dest.A = 255;
         }
 
         /// <inheritdoc/>
         public void ToArgb32(ref Argb32 dest) {
-            dest.Rgb = this;
+            dest.R = this.R;
+            dest.G = this.G;
+            dest.B = this.B;
             dest.A = 255;
         }
 

src/ImageSharp/PixelFormats/Rgba32.cs

@@ -22,24 +22,24 @@ namespace SixLabors.ImageSharp.PixelFormats
     public partial struct Rgba32 : IPixel<Rgba32>, IPackedVector<uint>
     {
         /// <summary>
-        /// Gets or sets the alpha component.
+        /// Gets or sets the red component.
         /// </summary>
-        public byte A;
+        public byte R;
 
         /// <summary>
-        /// Gets or sets the blue component.
+        /// Gets or sets the green component.
         /// </summary>
-        public byte B;
+        public byte G;
 
         /// <summary>
-        /// Gets or sets the green component.
+        /// Gets or sets the blue component.
         /// </summary>
-        public byte G;
+        public byte B;
 
         /// <summary>
-        /// Gets or sets the red component.
+        /// Gets or sets the alpha component.
         /// </summary>
-        public byte R;
+        public byte A;
 
         /// <summary>
         /// The shift count for the red component
@@ -174,22 +174,20 @@ namespace SixLabors.ImageSharp.PixelFormats
         }
 
         /// <summary>
-        /// Gets or sets the RGB components of this struct as <see cref="Rgb24"/>.
+        /// Gets or sets the RGB components of this struct as <see cref="Rgb24"/>
         /// </summary>
         public Rgb24 Rgb
         {
             [MethodImpl(MethodImplOptions.AggressiveInlining)]
             get
             {
-                return new Rgb24(this.R, this.G, this.B);
+                return Unsafe.As<Rgba32, Rgb24>(ref this);
             }
 
             [MethodImpl(MethodImplOptions.AggressiveInlining)]
             set
             {
-                this.R = value.R;
+                Unsafe.As<Rgba32, Rgb24>(ref this) = value;
-                this.G = value.G;
-                this.B = value.B;
             }
         }
 

tests/ImageSharp.Tests/PixelFormats/PackedPixelTests.cs

@@ -106,7 +106,7 @@ namespace SixLabors.ImageSharp.Tests.Colors
             float z = +0.5f;
             float w = -0.7f;
             var argb = new Argb32(x, y, z, w);
-            Assert.Equal(0x001a0080u, argb.PackedValue);
+            Assert.Equal(0x80001a00u, argb.PackedValue);
 
             // Test ordering
             var rgb = default(Rgb24);
@@ -925,7 +925,7 @@ namespace SixLabors.ImageSharp.Tests.Colors
             float z = +0.5f;
             float w = -0.7f;
             var rgba32 = new Rgba32(x, y, z, w);
-            Assert.Equal(0x1a008000u, rgba32.PackedValue);
+            Assert.Equal(0x80001Au, rgba32.PackedValue);
 
             // Test ordering
             var rgb = default(Rgb24);

tests/ImageSharp.Tests/PixelFormats/Rgb24Tests.cs

@@ -30,9 +30,9 @@ namespace SixLabors.ImageSharp.Tests
             var color = new Rgb24(1, 2, 3);
             byte* ptr = (byte*)&color;
 
-            Assert.Equal(3, ptr[0]);
+            Assert.Equal(1, ptr[0]);
             Assert.Equal(2, ptr[1]);
-            Assert.Equal(1, ptr[2]);
+            Assert.Equal(3, ptr[2]);
         }
 
         [Theory]