Re: PublicationActions - use bit flags.

Peter Smith <smithpb2250(at)gmail(dot)com> writes:
> On Thu, Dec 30, 2021 at 3:30 AM Justin Pryzby <pryzby(at)telsasoft(dot)com> wrote:
>> + if (pubform->pubinsert) pub->pubactions |= PUBACTION_INSERT;
>> This is usually written like:
>> pub->pubactions |= (pubform->pubinsert ? PUBACTION_INSERT : 0)

> Thanks for the info, I've modified those assignment styles as suggested.

FWIW, I think it's utter nonsense to claim that the second way is
preferred over the first. There may be some people who think
the second way is more legible, but I don't; and I'm pretty sure
that the first way is significantly more common in the PG codebase.

regards, tom lane