Thread: bms_prev_member won't work correctly if bitmapword is 64-bits

bms_prev_member won't work correctly if bitmapword is 64-bits

From
David Rowley
Date:
bms_prev_member mistakenly has a hardcoded 24 in the function. This
should really be BITS_PER_BITMAPWORD - 8 so that it is properly set to
56 if someone compiles with 64-bit bitmapwords.

The attached fixes this and also adds a test to exercise the function
a bit. [1] indicates there's currently no coverage of this function at
all.

[1] https://coverage.postgresql.org/src/backend/nodes/bitmapset.c.gcov.html

-- 
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

Attachment

Re: bms_prev_member won't work correctly if bitmapword is 64-bits

From
Teodor Sigaev
Date:
Thank you, pushed

David Rowley wrote:
> bms_prev_member mistakenly has a hardcoded 24 in the function. This
> should really be BITS_PER_BITMAPWORD - 8 so that it is properly set to
> 56 if someone compiles with 64-bit bitmapwords.
> 
> The attached fixes this and also adds a test to exercise the function
> a bit. [1] indicates there's currently no coverage of this function at
> all.
> 
> [1] https://coverage.postgresql.org/src/backend/nodes/bitmapset.c.gcov.html
> 

-- 
Teodor Sigaev                                   E-mail: teodor@sigaev.ru
                                                    WWW: http://www.sigaev.ru/


Re: bms_prev_member won't work correctly if bitmapword is 64-bits

From
David Rowley
Date:
On 24 April 2018 at 03:00, Teodor Sigaev <teodor@sigaev.ru> wrote:
> Thank you, pushed

Thanks!

-- 
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services