Hi Branden,

On Wed, Oct 08, 2025 at 01:54:27AM -0500, G. Branden Robinson wrote:
> > I've split it into a few patches.  One replacing array_length(),
> > another replacing SIZEOF(), and another replacing raw sizeof
> > divisions.
> 
> Thanks for the catching the ones I missed when migrating to
> `array_length()`.

You're welcome!

I have another patch that changes more cases, but I'm holding that one
because I found a bug in gnulib.  I've already reported it, and
hopefully I'll be able to send it next week or so.  You can apply the
current patches now, which are fine; the bug doesn't affect them.

> > I've made sure it builds, but didn't test the binary (I assume it
> > should work; the changes are too obvious).
> 
> Did you run "make check"?  We don't have 100% test coverage, but what we
> have is a lot better than nothing.

Nah; I'll let you run it for me this time.  I'll try to remember to run
it next time.  :)


Cheers,
Alex

-- 
<https://www.alejandro-colomar.es>
Use port 80 (that is, <...:80/>).

Attachment: signature.asc
Description: PGP signature

Reply via email to