[2022/11/02]

* Fixed a bug in the AciColor.FromHsl method, under certain circumstances the RGB values where not calculated correctly.
haplokuon · Nov 2, 2022 · c25b08e · c25b08e
1 parent 922fb00
commit c25b08e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/TestDxfDocument/Program.cs b/TestDxfDocument/Program.cs
@@ -29,8 +29,7 @@ namespace TestDxfDocument
     /// </summary>
     public class Program
     {
-
-        public static void Main()
+       public static void Main()
         {
             DxfDocument doc = Test(@"sample.dxf"); 
 

diff --git a/doc/Changelog.txt b/doc/Changelog.txt
@@ -1,5 +1,8 @@
 ## Change history
 
+### [2022/11/02]
+* Fixed a bug in the AciColor.FromHsl method, under certain circumstances the RGB values where not calculated correctly.
+
 ### [2022/11/01]
 * Workaround for bad DXF files that contain polyface with invalid geometry, they will be removed. 
 * Workaround for bad DXF files that contains entities without handles. This may derive in unexpected side effects.

diff --git a/netDxf/AciColor.cs b/netDxf/AciColor.cs
@@ -670,7 +670,7 @@ public static AciColor FromHsl(double hue, double saturation, double lightness)
                 double mid2 = v - vsf;
                 switch (sextant)
                 {
-                    case 0 | 6:
+                    case 0:
                         red = v;
                         green = mid1;
                         blue = m;
@@ -700,6 +700,11 @@ public static AciColor FromHsl(double hue, double saturation, double lightness)
                         green = m;
                         blue = mid2;
                         break;
+                    case 6:
+                        red = v;
+                        green = mid1;
+                        blue = m;
+                        break;
                 }
             }
             return new AciColor(red, green, blue);