From 499667eab5010791d48cb19172a2501dae1bea16 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Matthias=20Mail=C3=A4nder?= Date: Tue, 30 Jun 2015 22:17:26 +0200 Subject: [PATCH] remove unnecessary lock object --- .../Widgets/ColorMixerWidget.cs | 29 +++++++++---------- 1 file changed, 13 insertions(+), 16 deletions(-) diff --git a/OpenRA.Mods.Common/Widgets/ColorMixerWidget.cs b/OpenRA.Mods.Common/Widgets/ColorMixerWidget.cs index 394ad368bc..1a2dbdaab7 100644 --- a/OpenRA.Mods.Common/Widgets/ColorMixerWidget.cs +++ b/OpenRA.Mods.Common/Widgets/ColorMixerWidget.cs @@ -98,26 +98,23 @@ namespace OpenRA.Mods.Common.Widgets hue = H; } - lock (back) + unsafe { - unsafe + // Generate palette in HSV + fixed (byte* cc = &back[0]) { - // Generate palette in HSV - fixed (byte* cc = &back[0]) - { - var c = (int*)cc; - for (var v = 0; v < 256; v++) - for (var s = 0; s < 256; s++) - *(c + (v * 256) + s) = HSLColor.FromHSV(hue, s / 255f, (255 - v) / 255f).RGB.ToArgb(); - } + var c = (int*)cc; + for (var v = 0; v < 256; v++) + for (var s = 0; s < 256; s++) + *(c + (v * 256) + s) = HSLColor.FromHSV(hue, s / 255f, (255 - v) / 255f).RGB.ToArgb(); } + } - lock (bufferSync) - { - var swap = front; - front = back; - back = swap; - } + lock (bufferSync) + { + var swap = front; + front = back; + back = swap; } } }