comment

Author: ArrayBolt3

src/kloak.c

@@ -1079,6 +1079,11 @@ static int32_t get_ticks_from_scroll_accum(double *accum_ptr) {
   if (fpclassify(scroll_accum) != FP_ZERO) {
     assert(isfinite(scroll_accum));
     scroll_ticks_d = scroll_accum / SCROLL_UNITS_PER_TICK_D;
+    /* We intentionally use SCROLL_UNITS_PER_TICK here, not
+     * SCROLL_UNITS_PER_TICK_D. The extra fraction of a scroll tick we would
+     * gain from using SCROLL_UNITS_PER_TICK_D is not worthwhile, and having
+     * the enforced threshold slightly lower than the real threshold is
+     * arguably safer. */
     assert(scroll_ticks_d <= (INT32_MAX / SCROLL_UNITS_PER_TICK));
     assert(scroll_ticks_d >= (INT32_MIN / SCROLL_UNITS_PER_TICK));
     scroll_ticks = (int32_t)(scroll_ticks_d);