Merge pull request #285 from rubensr/jpeg-decode-grayscale

Adding simple decode for jpeg grayscale colorspaces. Also added test …

Former-commit-id: 16b48d25e6920474c969a47085e07f34f4bc7ffd
Former-commit-id: 01dcbe05f808a1cf3a2d8f07b76cf37cf31426e5
Former-commit-id: a23e88deacf8f7a42ea5e6958f3168b74d44f4c3

src/ImageProcessor/Formats/Jpg/JpegDecoder.cs

@@ -102,15 +102,34 @@ namespace ImageProcessor.Formats
 
             float[] pixels = new float[pixelWidth * pixelHeight * 4];
 
-            if (!(jpg.Colorspace == Colorspace.RGB && jpg.BitsPerComponent == 8))
+            if (jpg.Colorspace == Colorspace.RGB && jpg.BitsPerComponent == 8)
             {
-                throw new NotSupportedException("JpegDecoder only support RGB color space.");
-            }
+                Parallel.For(
+                    0,
+                    pixelHeight,
+                    y =>
+                        {
+                            SampleRow row = jpg.GetRow(y);
+
+                            for (int x = 0; x < pixelWidth; x++)
+                            {
+                                Sample sample = row.GetAt(x);
 
-            Parallel.For(
-                0,
-                pixelHeight,
-                y =>
+                                int offset = ((y * pixelWidth) + x) * 4;
+
+                                pixels[offset + 0] = sample[0] / 255f;
+                                pixels[offset + 1] = sample[1] / 255f;
+                                pixels[offset + 2] = sample[2] / 255f;
+                                pixels[offset + 3] = 1;
+                            }
+                        });
+            }
+            else if (jpg.Colorspace == Colorspace.Grayscale && jpg.BitsPerComponent == 8)
+            {
+                Parallel.For(
+                    0,
+                    pixelHeight,
+                    y =>
                     {
                         SampleRow row = jpg.GetRow(y);
 
@@ -121,11 +140,16 @@ namespace ImageProcessor.Formats
                             int offset = ((y * pixelWidth) + x) * 4;
 
                             pixels[offset + 0] = sample[0] / 255f;
-                            pixels[offset + 1] = sample[1] / 255f;
-                            pixels[offset + 2] = sample[2] / 255f;
+                            pixels[offset + 1] = sample[0] / 255f;
+                            pixels[offset + 2] = sample[0] / 255f;
                             pixels[offset + 3] = 1;
                         }
                     });
+            }
+            else
+            {
+                throw new NotSupportedException("JpegDecoder only supports RGB and Grayscale color spaces.");
+            }
 
             image.SetPixels(pixelWidth, pixelHeight, pixels);
         }

tests/ImageProcessor.Tests/Processors/ProcessorTestBase.cs

@@ -19,6 +19,7 @@ namespace ImageProcessor.Tests
         /// </summary>
         public static readonly List<string> Files = new List<string>
         {
+            "TestImages/Formats/Jpg/Floorplan.jpeg",
             "TestImages/Formats/Jpg/Calliphora.jpg",
             //"TestImages/Formats/Jpg/gamma_dalai_lama_gray.jpg", //Perf: Enable for local testing only
             "TestImages/Formats/Bmp/Car.bmp",

tests/ImageProcessor.Tests/TestImages/Formats/Jpg/Floorplan.jpeg.REMOVED.git-id

@@ -0,0 +1 @@
+5a1eaf806b7f3793d3db41c85bac6366584bce83