Browse Source

Address review comments

js/color-alpha-handling
Amaury Levé 5 years ago
parent
commit
e687f7b10a
  1. 4
      src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs
  2. 5
      src/ImageSharp/Processing/Processors/Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs

4
src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs

@ -17,7 +17,7 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
/// Applies the swizzle transformation to a given point.
/// </summary>
/// <param name="point">Point to transform.</param>
/// <param name="newPoint">The transformed point.</param>
void Transform(Point point, out Point newPoint);
/// <returns>The transformed point.</returns>
Point Transform(Point point);
}
}

5
src/ImageSharp/Processing/Processors/Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs

@ -29,10 +29,11 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
Point newPoint;
for (p.Y = 0; p.Y < source.Height; p.Y++)
{
Span<TPixel> rowSpan = source.GetPixelRowSpan(p.Y);
for (p.X = 0; p.X < source.Width; p.X++)
{
this.swizzler.Transform(p, out newPoint);
destination[newPoint.X, newPoint.Y] = source[p.X, p.Y];
newPoint = this.swizzler.Transform(p);
destination[newPoint.X, newPoint.Y] = rowSpan[p.X];
}
}
}

Loading…
Cancel
Save