On Mon, Mar 3, 2025 at 11:21 PM Nathan Bossart <nathandbossart@gmail.com> wrote:
>
> On Mon, Mar 03, 2025 at 05:58:43PM +0700, John Naylor wrote:
> True. One small thing we could do is to require "found_objs" (the double
> pointer) to always be non-NULL, but that just compels some callers to
> provide otherwise-unused variables. I've left the interface alone for now.
One thing to consider is that a pointer named "dummy" is self-documenting.
> > That's better, and if we end up with this interface, we'll want quotes
> > around the names so the eye can tell where the "*" belong.
>
> I did that in v3. I also tried to break up this comment into bullet points
> for better separation and logical flow.
That's much easier to follow, thanks.
--
John Naylor
Amazon Web Services
