> > While looking at something nearby, I noticed a naming inconsistency for
> > the parameter name in LWLockNewTrancheId(). It should be tranche_name,
> > similar to RequestNamedLWLockTranche(), and in other places it's
> > referenced in the docs/comments.
>
> I don't mind changing it, but I don't find "tranche_name" to be
> substantially better than "name" in this case.

It's just a matter of consistency and it's more descriptive to the
caller, IMO.

--
Sami Imseih
Amazon Web Services (AWS)


Reply via email to