From 32bc47405522efbb8a809700c96be37f0236018e Mon Sep 17 00:00:00 2001
From: Frank <91616163+softhack007@users.noreply.github.com>
Date: Wed, 4 Mar 2026 00:17:43 +0100
Subject: [PATCH] spots effect bugfix, take 2

next try: make sure that the strip is filled, and all gaps are equal
---
 wled00/FX.cpp | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/wled00/FX.cpp b/wled00/FX.cpp
index 0d57af3f..471f6f5f 100644
--- a/wled00/FX.cpp
+++ b/wled00/FX.cpp
@@ -3015,20 +3015,15 @@ uint16_t spots_base(uint16_t threshold)
   if (SEGLEN <= 1) return mode_oops();
   if (!SEGMENT.check2) SEGMENT.fill(SEGCOLOR(1));
 
-  // constants for fixed point scaling
-  constexpr uint8_t  ZONELEN_FP_SHIFT = 3;
-  constexpr uint32_t ZONELEN_FP_SCALE = 1U << ZONELEN_FP_SHIFT;
-
-  unsigned maxZones = max(1, SEGLEN >> 2); // prevents "0 zones"
-  unsigned zones    = 1U + ((uint32_t(SEGMENT.intensity) * maxZones) >> 8);
-  unsigned zoneLen  =  uint32_t(SEGLEN) / zones;
-  unsigned zoneLen8 = (uint32_t(SEGLEN) * ZONELEN_FP_SCALE) / zones; // zoneLength * 8 (fixed‑point) -> avoids gaps at right/left sides
-  unsigned offset   = (uint32_t(SEGLEN) - ((zones * zoneLen8) >> ZONELEN_FP_SHIFT)) >> 1;
+  unsigned maxZones = max(1, SEGLEN / 4); // prevents "0 zones"
+  int zones         = 1U + ((uint32_t(SEGMENT.intensity) * maxZones + 127) >> 8); // with rounding
+  unsigned zoneLen  = (uint32_t(SEGLEN) + zones-1) / zones;  // round up (ceil)
+  int offset = ((int)SEGLEN - (zones * zoneLen)) / 2; // center the zones on the segment (can not use bit shift on negative number)
 
   for (unsigned z = 0; z < zones; z++)
   {
-    unsigned pos = offset + ((z * zoneLen8) >> ZONELEN_FP_SHIFT);
-    for (unsigned i = 0; i < zoneLen; i++)
+    int pos = offset + (z * zoneLen);
+    for (int i = 0; i < zoneLen; i++)
     {
       unsigned wave = triwave16((i * 0xFFFF) / zoneLen);
       if (wave > threshold) {