On Tue, Sep 15, 2020 at 9:56 AM Thomas Munro <thomas.mu...@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.