On Tue, Sep 15, 2020 at 9:56 AM Thomas Munro <thomas.munro@gmail.com> wrote:
> > I looked over the patch and the only thing I saw was that we might
> > also want to remove the following line:
> >
> > #define DEF_FFACTOR 1 /* default fill factor */
>
> Right, thanks. Fixed in the attached.
Pushed. Thanks Jakub, everyone.
Separately, we really should tidy up the int/long/uint32/size_t
confusion in this module. I know we have K&R C-era long-itude in
numerous other modules, but this one is a little more egregious in its
data type mixing.