From e8b59c6bb4b8f77714170285ef7710b4c0e4e6db Mon Sep 17 00:00:00 2001
From: Dmitry Pentin <riotbr3aker@gmail.com>
Date: Sat, 27 Nov 2021 10:11:41 +0300
Subject: [PATCH] Fixed access modifier & docs

---
 .../Jpeg/Components/Decoder/IRawJpegData.cs   |  1 -
 .../Decoder/JpegComponentPostProcessor.cs     | 52 +++++++++----------
 2 files changed, 25 insertions(+), 28 deletions(-)

diff --git a/src/ImageSharp/Formats/Jpeg/Components/Decoder/IRawJpegData.cs b/src/ImageSharp/Formats/Jpeg/Components/Decoder/IRawJpegData.cs
index 0b80acc5d..33815e539 100644
--- a/src/ImageSharp/Formats/Jpeg/Components/Decoder/IRawJpegData.cs
+++ b/src/ImageSharp/Formats/Jpeg/Components/Decoder/IRawJpegData.cs
@@ -5,7 +5,6 @@ using System;
 
 namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder
 {
-    /// <inheritdoc />
     /// <summary>
     /// Represents decompressed, unprocessed jpeg data with spectral space <see cref="IJpegComponent" />-s.
     /// </summary>
diff --git a/src/ImageSharp/Formats/Jpeg/Components/Decoder/JpegComponentPostProcessor.cs b/src/ImageSharp/Formats/Jpeg/Components/Decoder/JpegComponentPostProcessor.cs
index 35f421263..b87d74720 100644
--- a/src/ImageSharp/Formats/Jpeg/Components/Decoder/JpegComponentPostProcessor.cs
+++ b/src/ImageSharp/Formats/Jpeg/Components/Decoder/JpegComponentPostProcessor.cs
@@ -21,6 +21,21 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder
         /// </summary>
         private readonly JpegFrame frame;
 
+        /// <summary>
+        /// Gets the maximal number of block rows being processed in one step.
+        /// </summary>
+        private readonly int blockRowsPerStep;
+
+        /// <summary>
+        /// Gets the <see cref="IJpegComponent"/> component containing decoding meta information.
+        /// </summary>
+        private readonly IJpegComponent component;
+
+        /// <summary>
+        /// Gets the <see cref="IRawJpegData"/> instance containing decoding meta information.
+        /// </summary>
+        private readonly IRawJpegData rawJpeg;
+
         /// <summary>
         /// Initializes a new instance of the <see cref="JpegComponentPostProcessor"/> class.
         /// </summary>
@@ -28,39 +43,22 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder
         {
             this.frame = frame;
 
-            this.Component = component;
-            this.RawJpeg = rawJpeg;
-            this.blockAreaSize = this.Component.SubSamplingDivisors * 8;
+            this.component = component;
+            this.rawJpeg = rawJpeg;
+            this.blockAreaSize = this.component.SubSamplingDivisors * 8;
             this.ColorBuffer = memoryAllocator.Allocate2DOveraligned<float>(
                 postProcessorBufferSize.Width,
                 postProcessorBufferSize.Height,
                 this.blockAreaSize.Height);
 
-            this.BlockRowsPerStep = postProcessorBufferSize.Height / 8 / this.Component.SubSamplingDivisors.Height;
+            this.blockRowsPerStep = postProcessorBufferSize.Height / 8 / this.component.SubSamplingDivisors.Height;
         }
 
-        public IRawJpegData RawJpeg { get; }
-
-        /// <summary>
-        /// Gets the <see cref="Component"/>
-        /// </summary>
-        public IJpegComponent Component { get; }
-
         /// <summary>
         /// Gets the temporary working buffer of color values.
         /// </summary>
         public Buffer2D<float> ColorBuffer { get; }
 
-        /// <summary>
-        /// Gets <see cref="IJpegComponent.SizeInBlocks"/>
-        /// </summary>
-        public Size SizeInBlocks => this.Component.SizeInBlocks;
-
-        /// <summary>
-        /// Gets the maximal number of block rows being processed in one step.
-        /// </summary>
-        public int BlockRowsPerStep { get; }
-
         /// <inheritdoc />
         public void Dispose() => this.ColorBuffer.Dispose();
 
@@ -69,20 +67,20 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder
         /// </summary>
         public void CopyBlocksToColorBuffer(int spectralStep)
         {
-            Buffer2D<Block8x8> spectralBuffer = this.Component.SpectralBlocks;
+            Buffer2D<Block8x8> spectralBuffer = this.component.SpectralBlocks;
 
             float maximumValue = this.frame.MaxColorChannelValue;
 
             int destAreaStride = this.ColorBuffer.Width;
 
-            int yBlockStart = spectralStep * this.BlockRowsPerStep;
+            int yBlockStart = spectralStep * this.blockRowsPerStep;
 
-            Size subSamplingDivisors = this.Component.SubSamplingDivisors;
+            Size subSamplingDivisors = this.component.SubSamplingDivisors;
 
-            Block8x8F dequantTable = this.RawJpeg.QuantizationTables[this.Component.QuantizationTableIndex];
+            Block8x8F dequantTable = this.rawJpeg.QuantizationTables[this.component.QuantizationTableIndex];
             Block8x8F workspaceBlock = default;
 
-            for (int y = 0; y < this.BlockRowsPerStep; y++)
+            for (int y = 0; y < this.blockRowsPerStep; y++)
             {
                 int yBuffer = y * this.blockAreaSize.Height;
 
@@ -118,7 +116,7 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder
 
         public void ClearSpectralBuffers()
         {
-            Buffer2D<Block8x8> spectralBlocks = this.Component.SpectralBlocks;
+            Buffer2D<Block8x8> spectralBlocks = this.component.SpectralBlocks;
             for (int i = 0; i < spectralBlocks.Height; i++)
             {
                 spectralBlocks.GetRowSpan(i).Clear();