Re: Change definitions of bitmap flags to bit-shifting style - Mailing list pgsql-hackers

From Laurenz Albe
Subject Re: Change definitions of bitmap flags to bit-shifting style
Date
Msg-id 0e8cde37d304ebbae582455aad145f42757f1f89.camel@cybertec.at
Whole thread Raw
In response to Re: Change definitions of bitmap flags to bit-shifting style  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Sat, 2020-12-05 at 13:03 -0500, Tom Lane wrote:
> Peter Eisentraut <peter.eisentraut@enterprisedb.com> writes:
> 
> > The attached patch changes definitions like
> >       #define FOO 0x01
> >       #define BAR 0x02
> > to
> >       #define FOO (1 << 0)
> >       #define BAR (1 << 1)
> > etc.
> 
> > Both styles are currently in use, but the latter style seems more 
> > readable and easier to update.
> 
> FWIW, personally I'd vote for doing the exact opposite.  When you are
> debugging and examining the contents of a bitmask variable, it's easier to
> correlate a value like "0x03" with definitions made in the former style.
> Or at least I think so; maybe others see it differently.

+1

Laurenz Albe




pgsql-hackers by date:

Previous
From: Andy Fan
Date:
Subject: Re: [PATCH] Keeps tracking the uniqueness with UniqueKey
Next
From: Michael Paquier
Date:
Subject: Re: Change definitions of bitmap flags to bit-shifting style