From c3b7214a1927966ee6cbe3faca72f0487fa11c22 Mon Sep 17 00:00:00 2001
From: Andreas Fabri <andreas.fabri@geometryfactory.com>
Date: Wed, 26 Jul 2023 15:44:33 +0100
Subject: [PATCH] Fix the fix.  The floats were already in the range [0,256]

---
 .../examples/Arrangement_on_surface_2/draw_arr.cpp       | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/Arrangement_on_surface_2/examples/Arrangement_on_surface_2/draw_arr.cpp b/Arrangement_on_surface_2/examples/Arrangement_on_surface_2/draw_arr.cpp
index b62ba6aadc1..8c3c9b4daf8 100644
--- a/Arrangement_on_surface_2/examples/Arrangement_on_surface_2/draw_arr.cpp
+++ b/Arrangement_on_surface_2/examples/Arrangement_on_surface_2/draw_arr.cpp
@@ -18,7 +18,7 @@ using Arrangement_2 = CGAL::Arrangement_2<Traits>;
  * \param value Value component range: [0, 1]
  * \return tuple<red, green, blue>, where each component is in the range [0, 255]
 */
-std::tuple<float, float, float>
+std::tuple<unsigned char, unsigned char, unsigned char>
 hsv_to_rgb(float hue, float sat, float value) {
   float red, green, blue;
   float fc = value * sat; // Chroma
@@ -69,7 +69,10 @@ hsv_to_rgb(float hue, float sat, float value) {
   red *= 255;
   green *= 255;
   blue *= 255;
-  return std::make_tuple(red, green, blue);
+  unsigned char redc = (unsigned char)red;
+  unsigned char greenc = (unsigned char)green;
+  unsigned char bluec = (unsigned char)blue;
+  return std::make_tuple(redc, greenc, bluec);
 }
 
 int main() {
@@ -104,7 +107,7 @@ int main() {
                     float r, g, b;
                     typedef unsigned char uchar;
                     std::tie(r, g, b) = hsv_to_rgb(h, s, v);
-                    return CGAL::IO::Color(uchar(r*255), uchar(g*255), uchar(b*255));
+                    return CGAL::IO::Color(r,g,b);
                   }, "hsv colors", true);
 
   return EXIT_SUCCESS;