[m-rev.] for post-commit review: uints in edit_seq.m

Julien Fischer jfischer at opturion.com
Thu Jan 1 18:11:43 AEDT 2026


On Thu, 1 Jan 2026 at 18:01, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:

> > On Thu, 1 Jan 2026 at 17:22, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:
> > >
> > > The concept has been agreed and the diff itself is straightforward.
> > > The only part worth reviewing in the entry in NEWS.md. I would also
> > > like to know if anyone objects to adding list.ulength, a version of
> > > list.length that returns a uint, or if anyone has a better name for it.
> >
> > No objections from me, I have thought about adding myself in the past.
> > If we are going to go down this road, then map.count, set.count etc should
> > also have versions returning uints (e.g. ucount).
>
> Will do. I presume you are ok with the naming scheme of putting a "u" in front
> of the predicate and function names?

Yes, there needs to be a consistent naming convention for these new
predicates and functions.  Prefixing the unsigned variant with "u" is fine
by me.

Julien.


More information about the reviews mailing list