Browse Source

Removed unnecessary XML comments

pull/1108/head
Sergio Pedri 6 years ago
parent
commit
2bc5d82fbb
  1. 31
      src/ImageSharp/Processing/Processors/Convolution/BokehBlurProcessor{TPixel}.cs
  2. 10
      src/ImageSharp/Processing/Processors/Convolution/Convolution2DProcessor{TPixel}.cs
  3. 9
      src/ImageSharp/Processing/Processors/Convolution/Convolution2PassProcessor{TPixel}.cs
  4. 9
      src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor{TPixel}.cs
  5. 9
      src/ImageSharp/Processing/Processors/Convolution/EdgeDetectorCompassProcessor{TPixel}.cs
  6. 23
      src/ImageSharp/Processing/Processors/Transforms/AffineTransformProcessor{TPixel}.cs

31
src/ImageSharp/Processing/Processors/Convolution/BokehBlurProcessor{TPixel}.cs

@ -337,13 +337,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly Buffer2D<TPixel> sourcePixels;
private readonly Complex64[] kernel;
/// <summary>
/// Initializes a new instance of the <see cref="ApplyVerticalConvolutionRowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetValues">The target <see cref="ComplexVector4"/> values to use to store the results.</param>
/// <param name="sourcePixels">The source pixels. Cannot be null.</param>
/// <param name="kernel">The 1D kernel.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public ApplyVerticalConvolutionRowIntervalAction(
ref Rectangle bounds,
@ -388,15 +381,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly float z;
private readonly float w;
/// <summary>
/// Initializes a new instance of the <see cref="ApplyHorizontalConvolutionRowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetValues">The target <see cref="Vector4"/> values to use to store the results.</param>
/// <param name="sourceValues">The source complex values. Cannot be null.</param>
/// <param name="kernel">The 1D kernel.</param>
/// <param name="z">The weight factor for the real component of the complex pixel values.</param>
/// <param name="w">The weight factor for the imaginary component of the complex pixel values.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public ApplyHorizontalConvolutionRowIntervalAction(
ref Rectangle bounds,
@ -443,13 +427,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly Configuration configuration;
private readonly float gamma;
/// <summary>
/// Initializes a new instance of the <see cref="ApplyGammaExposureRowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetPixels">The target pixel buffer to adjust.</param>
/// <param name="configuration">The <see cref="Configuration"/></param>
/// <param name="gamma">The gamma parameter to use.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public ApplyGammaExposureRowIntervalAction(
Rectangle bounds,
@ -500,14 +477,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly Configuration configuration;
private readonly float inverseGamma;
/// <summary>
/// Initializes a new instance of the <see cref="ApplyInverseGammaExposureRowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetPixels">The target pixels to apply the process to.</param>
/// <param name="sourceValues">The source <see cref="Vector4"/> values. Cannot be null.</param>
/// <param name="configuration">The <see cref="Configuration"/></param>
/// <param name="inverseGamma">The inverse gamma parameter to use.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public ApplyInverseGammaExposureRowIntervalAction(
Rectangle bounds,

10
src/ImageSharp/Processing/Processors/Convolution/Convolution2DProcessor{TPixel}.cs

@ -87,16 +87,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly Configuration configuration;
private readonly bool preserveAlpha;
/// <summary>
/// Initializes a new instance of the <see cref="Convolution2DProcessor{TPixel}.RowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetPixels">The target pixel buffer to adjust.</param>
/// <param name="sourcePixels">The source pixels. Cannot be null.</param>
/// <param name="kernelY">The vertical kernel operator.</param>
/// <param name="kernelX">The horizontal kernel operator.</param>
/// <param name="configuration">The <see cref="Configuration"/></param>
/// <param name="preserveAlpha">Whether the convolution filter is applied to alpha as well as the color channels.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public RowIntervalAction(
Rectangle bounds,

9
src/ImageSharp/Processing/Processors/Convolution/Convolution2PassProcessor{TPixel}.cs

@ -88,15 +88,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly Configuration configuration;
private readonly bool preserveAlpha;
/// <summary>
/// Initializes a new instance of the <see cref="RowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetPixels">The target pixel buffer to adjust.</param>
/// <param name="sourcePixels">The source pixels. Cannot be null.</param>
/// <param name="kernel">The kernel operator.</param>
/// <param name="configuration">The <see cref="Configuration"/></param>
/// <param name="preserveAlpha">Whether the convolution filter is applied to alpha as well as the color channels.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public RowIntervalAction(
Rectangle bounds,

9
src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor{TPixel}.cs

@ -77,15 +77,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly Configuration configuration;
private readonly bool preserveAlpha;
/// <summary>
/// Initializes a new instance of the <see cref="ConvolutionProcessor{TPixel}.RowIntervalAction"/> struct.
/// </summary>
/// <param name="bounds">The target processing bounds for the current instance.</param>
/// <param name="targetPixels">The target pixel buffer to adjust.</param>
/// <param name="sourcePixels">The source pixels. Cannot be null.</param>
/// <param name="kernel">The kernel operator.</param>
/// <param name="configuration">The <see cref="Configuration"/></param>
/// <param name="preserveAlpha">Whether the convolution filter is applied to alpha as well as the color channels.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public RowIntervalAction(
Rectangle bounds,

9
src/ImageSharp/Processing/Processors/Convolution/EdgeDetectorCompassProcessor{TPixel}.cs

@ -121,15 +121,6 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
private readonly int shiftY;
private readonly int shiftX;
/// <summary>
/// Initializes a new instance of the <see cref="RowIntervalAction"/> struct.
/// </summary>
/// <param name="targetPixels">The target pixel buffer to adjust.</param>
/// <param name="passPixels">The processed pixels for the current iteration. Cannot be null.</param>
/// <param name="minX">The minimum horizontal offset.</param>
/// <param name="maxX">The maximum horizontal offset.</param>
/// <param name="shiftY">The vertical offset shift.</param>
/// <param name="shiftX">The horizontal offset shift.</param>
[MethodImpl(InliningOptions.ShortMethod)]
public RowIntervalAction(
Buffer2D<TPixel> targetPixels,

23
src/ImageSharp/Processing/Processors/Transforms/AffineTransformProcessor{TPixel}.cs

@ -17,8 +17,8 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
internal class AffineTransformProcessor<TPixel> : TransformProcessor<TPixel>
where TPixel : struct, IPixel<TPixel>
{
private Size targetSize;
private Matrix3x2 transformMatrix;
private readonly Size targetSize;
private readonly Matrix3x2 transformMatrix;
private readonly IResampler resampler;
/// <summary>
@ -58,26 +58,25 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
if (this.resampler is NearestNeighborResampler)
{
Rectangle sourceBounds = this.SourceRectangle;
var nearestRowAction = new NearestNeighborRowIntervalAction(ref sourceBounds, ref matrix, width, source, destination);
ParallelRowIterator.IterateRows(
targetBounds,
configuration,
in nearestRowAction);
new NearestNeighborRowIntervalAction(this.SourceRectangle, ref matrix, width, source, destination));
return;
}
using var kernelMap = new TransformKernelMap(configuration, source.Size(), destination.Size(), this.resampler);
var rowAction = new RowIntervalAction(configuration, kernelMap, ref matrix, width, source, destination);
ParallelRowIterator.IterateRows<RowIntervalAction, Vector4>(
targetBounds,
configuration,
in rowAction);
new RowIntervalAction(configuration, kernelMap, ref matrix, width, source, destination));
}
/// <summary>
/// A <see langword="struct"/> implementing the nearest neighbor resampler logic for <see cref="AffineTransformProcessor{T}"/>.
/// </summary>
private readonly struct NearestNeighborRowIntervalAction : IRowIntervalAction
{
private readonly Rectangle bounds;
@ -88,7 +87,7 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
[MethodImpl(InliningOptions.ShortMethod)]
public NearestNeighborRowIntervalAction(
ref Rectangle bounds,
Rectangle bounds,
ref Matrix3x2 matrix,
int maxX,
ImageFrame<TPixel> source,
@ -101,6 +100,8 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
this.destination = destination;
}
/// <inheritdoc/>
/// <param name="rows"></param>
[MethodImpl(InliningOptions.ShortMethod)]
public void Invoke(in RowInterval rows)
{
@ -120,6 +121,9 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
}
}
/// <summary>
/// A <see langword="struct"/> implementing the transformation logic for <see cref="AffineTransformProcessor{T}"/>.
/// </summary>
private readonly struct RowIntervalAction : IRowIntervalAction<Vector4>
{
private readonly Configuration configuration;
@ -146,6 +150,7 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
this.destination = destination;
}
/// <inheritdoc/>
[MethodImpl(InliningOptions.ShortMethod)]
public void Invoke(in RowInterval rows, Memory<Vector4> memory)
{

Loading…
Cancel
Save