Added docs & stylecop fixes

src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter420{TPixel}.cs

@@ -2,7 +2,6 @@
 // Licensed under the Apache License, Version 2.0.
 
 using System;
-using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 using SixLabors.ImageSharp.Advanced;
 using SixLabors.ImageSharp.PixelFormats;
@@ -66,13 +65,16 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
         /// </summary>
         private Span<Rgb24> rgbSpan;
 
-        // TODO: docs
+        /// <summary>
+        /// Sampled pixel buffer size
+        /// </summary>
         private Size samplingAreaSize;
 
-        // TODO: docs
+        /// <summary>
+        /// <see cref="Configuration"/> for internal operations
+        /// </summary>
         private Configuration config;
 
-
         public YCbCrForwardConverter420(ImageFrame<TPixel> frame)
         {
             // matrices would be filled during convert calls

src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter444{TPixel}.cs

@@ -15,16 +15,21 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
     internal ref struct YCbCrForwardConverter444<TPixel>
         where TPixel : unmanaged, IPixel<TPixel>
     {
-        // TODO: docs
+        /// <summary>
+        /// Number of pixels processed per single <see cref="Convert(int, int, ref RowOctet{TPixel})"/> call
+        /// </summary>
         private const int PixelsPerSample = 8 * 8;
 
-        // TODO: docs
+        /// <summary>
+        /// Total byte size of processed pixels converted from TPixel to <see cref="Rgb24"/>
+        /// </summary>
         private const int RgbSpanByteSize = PixelsPerSample * 3;
 
-        // TODO: docs
+        /// <summary>
+        /// <see cref="Size"/> of sampling area from given frame pixel buffer
+        /// </summary>
         private static readonly Size SampleSize = new Size(8, 8);
 
-
         /// <summary>
         /// The Y component
         /// </summary>
@@ -55,11 +60,15 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
         /// </summary>
         private Span<Rgb24> rgbSpan;
 
-        // TODO: docs
+        /// <summary>
+        /// Sampled pixel buffer size
+        /// </summary>
         private Size samplingAreaSize;
 
-        // TODO: docs
+        /// <summary>
-        private readonly Configuration config;
+        /// <see cref="Configuration"/> for internal operations
+        /// </summary>
+        private Configuration config;
 
         public YCbCrForwardConverter444(ImageFrame<TPixel> frame)
         {

src/ImageSharp/Formats/Jpeg/Components/FastFloatingPointDCT.cs

@@ -273,7 +273,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components
         /// <summary>
         /// Apply floating point FDCT from src into dest
         /// </summary>
-        /// <remarks></remarks>
         /// <param name="src">Source</param>
         /// <param name="dest">Destination</param>
         /// <param name="temp">Temporary block provided by the caller for optimization</param>
@@ -467,7 +466,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components
             Vector256<float> mb1 = Avx.Add(SimdUtils.HwIntrinsics.MultiplyAdd(mz1, my3, C_V_3_0727), mz2);
             Vector256<float> mb0 = Avx.Add(SimdUtils.HwIntrinsics.MultiplyAdd(mz0, my1, C_V_1_5013), mz3);
 
-
             Vector256<float> my2 = s.V2;
             Vector256<float> my6 = s.V6;
             mz4 = Avx.Multiply(Avx.Add(my2, my6), C_V_0_5411);

src/ImageSharp/Formats/Jpeg/JpegEncoderCore.cs

@@ -5,14 +5,11 @@ using System;
 using System.Buffers.Binary;
 using System.IO;
 using System.Linq;
-using System.Runtime.CompilerServices;
-using System.Runtime.InteropServices;
 using System.Threading;
 using SixLabors.ImageSharp.Common.Helpers;
 using SixLabors.ImageSharp.Formats.Jpeg.Components;
 using SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder;
 using SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder;
-using SixLabors.ImageSharp.Memory;
 using SixLabors.ImageSharp.Metadata;
 using SixLabors.ImageSharp.Metadata.Profiles.Exif;
 using SixLabors.ImageSharp.Metadata.Profiles.Icc;
@@ -69,7 +66,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg
                 99, 99, 99, 99, 99, 99, 99, 99,
             };
 
-
         /// <summary>
         /// A scratch buffer to reduce allocations.
         /// </summary>
@@ -625,7 +621,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg
         private void WriteStartOfScan<TPixel>(Image<TPixel> image, int componentCount, CancellationToken cancellationToken)
             where TPixel : unmanaged, IPixel<TPixel>
         {
-            // TODO: Need a JpegScanEncoder<TPixel> class or struct that encapsulates the scan-encoding implementation. (Similar to JpegScanDecoder.)
             Span<byte> componentId = stackalloc byte[]
             {
                 0x01,

tests/ImageSharp.Tests/Formats/Jpg/DCTTests.cs

@@ -63,7 +63,6 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg
                 this.CompareBlocks(expected, actual, 1f);
             }
 
-
             // Inverse transform
             [Theory]
             [InlineData(1)]
@@ -182,7 +181,6 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg
                 Assert.Equal(actualDest, expectedDest, new ApproximateFloatComparer(1f));
             }
 
-
             // Forward transform
             [Theory]
             [InlineData(1)]