On Thu, May 23, 2024 at 05:26:42PM -0400, Taylor Blau wrote: > +static uint32_t pseudo_merge_group_size(const struct pseudo_merge_group *group, > + const struct pseudo_merge_matches *matches, > + uint32_t i) > +{ > + double C = 0.0f; This version mostly drops the "f" from floating point constants, since they're now doubles. But this one doesn't. I don't think it really matters in practice (the number obviously fits in a float, and then it ends up as a double in the variable), so it's really just a style / readability question. Not worth a re-roll IMHO. -Peff