From a5966b24e8999cd81f7c463b0d0f374f8f9aa183 Mon Sep 17 00:00:00 2001 From: James Jackson-South Date: Thu, 7 Jan 2016 20:24:35 +1100 Subject: [PATCH] Inline methods for perf Former-commit-id: e899dbeb1b67c44cb17cc1e73aca7241a19f1555 Former-commit-id: b0a8ab76cc5caa54b8c9646ac6999b3d1b26dd82 Former-commit-id: eb027298759c1c3be00ce2820ccbf5191a393883 --- src/ImageProcessor/Filters/Brightness.cs | 25 +++-------- src/ImageProcessor/Filters/Contrast.cs | 43 +++++++------------ src/ImageProcessor/Samplers/EntropyCrop.cs | 3 ++ .../Processors/Filters/FilterTests.cs | 1 - 4 files changed, 24 insertions(+), 48 deletions(-) diff --git a/src/ImageProcessor/Filters/Brightness.cs b/src/ImageProcessor/Filters/Brightness.cs index 8a6b935e9e..6a5ae3985d 100644 --- a/src/ImageProcessor/Filters/Brightness.cs +++ b/src/ImageProcessor/Filters/Brightness.cs @@ -50,28 +50,15 @@ namespace ImageProcessor.Filters { for (int x = startX; x < endX; x++) { - target[x, y] = AdjustBrightness(source[x, y], brightness); + Color color = Color.Expand(source[x, y]); + + Vector3 vector3 = color.ToVector3(); + vector3 += new Vector3(brightness); + + target[x, y] = Color.Compress(new Color(vector3, color.A)); } } }); } - - /// - /// Returns a with the brightness adjusted. - /// - /// The source color. - /// The brightness adjustment factor. - /// - /// The . - /// - private static Color AdjustBrightness(Color color, float brightness) - { - color = Color.Expand(color); - - Vector3 vector3 = color.ToVector3(); - vector3 += new Vector3(brightness, brightness, brightness); - - return Color.Compress(new Color(vector3, color.A)); - } } } diff --git a/src/ImageProcessor/Filters/Contrast.cs b/src/ImageProcessor/Filters/Contrast.cs index 72dd0b8f0f..4a45e2c481 100644 --- a/src/ImageProcessor/Filters/Contrast.cs +++ b/src/ImageProcessor/Filters/Contrast.cs @@ -49,38 +49,25 @@ namespace ImageProcessor.Filters { for (int x = startX; x < endX; x++) { - target[x, y] = AdjustContrast(source[x, y], contrast); - } - } - }); - } + Color color = Color.Expand(source[x, y]); - /// - /// Returns a with the contrast adjusted. - /// - /// The source color. - /// The contrast adjustment factor. - /// - /// The . - /// - private static Color AdjustContrast(Color color, float contrast) - { - color = Color.Expand(color); - - // Seems to be faster than Vector3. - color.R -= 0.5f; - color.R *= contrast; - color.R += 0.5f; + // Seems to be faster than Vector3. + color.R -= 0.5f; + color.R *= contrast; + color.R += 0.5f; - color.G -= 0.5f; - color.G *= contrast; - color.G += 0.5f; + color.G -= 0.5f; + color.G *= contrast; + color.G += 0.5f; - color.B -= 0.5f; - color.B *= contrast; - color.B += 0.5f; + color.B -= 0.5f; + color.B *= contrast; + color.B += 0.5f; - return Color.Compress(color); + target[x, y] = Color.Compress(color); + } + } + }); } } } diff --git a/src/ImageProcessor/Samplers/EntropyCrop.cs b/src/ImageProcessor/Samplers/EntropyCrop.cs index 0f1ce9067e..3ef7f04533 100644 --- a/src/ImageProcessor/Samplers/EntropyCrop.cs +++ b/src/ImageProcessor/Samplers/EntropyCrop.cs @@ -16,6 +16,9 @@ namespace ImageProcessor.Samplers /// public class EntropyCrop : ParallelImageProcessor { + /// + /// The rectangle for cropping + /// private Rectangle cropRectangle; /// diff --git a/tests/ImageProcessor.Tests/Processors/Filters/FilterTests.cs b/tests/ImageProcessor.Tests/Processors/Filters/FilterTests.cs index a448f20519..c3b26590cb 100644 --- a/tests/ImageProcessor.Tests/Processors/Filters/FilterTests.cs +++ b/tests/ImageProcessor.Tests/Processors/Filters/FilterTests.cs @@ -3,7 +3,6 @@ namespace ImageProcessor.Tests { using System.Diagnostics; using System.IO; - using System.Numerics; using ImageProcessor.Filters;