On Tue, Sep 07, 2021 at 02:59:58PM +0900, Michael Paquier wrote: > In 0002, everything is a boolean expression except for > SpGistPageStoresNulls() and GistPageIsLeaf(). So that's a good > cleanup overall.
I looked again at 0002 again yesterday, and that was an improvement for most of those locations, where we already use a boolean as expression, so done mostly as of fd0625c. > - pathnode->parallel_aware = parallel_workers > 0 ? true : false; > + pathnode->parallel_aware = parallel_workers > 0; > I also prefer that we keep the parenthesis for such things. That's > more readable and easier to reason about. Adjusted these as well. -- Michael
signature.asc
Description: PGP signature