[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