Hi, On 2020-12-07 17:25:41 -0500, Tom Lane wrote: > Fair enough. It wasn't entirely clear to me whether it'd be kosher to > write > EEO_CASE(EEOP_SBSREF_OLD) > EEO_CASE(EEOP_SBSREF_ASSIGN) > EEO_CASE(EEOP_SBSREF_FETCH) > { > // do something > EEO_NEXT(); > } > > I can see that that should work for the two existing implementations > of EEO_CASE, but I wasn't sure if you wanted to wire in an assumption > that it'll always work.
I don't think it's likely to be a problem, and if it ends up being one, we can still deduplicate the ops at that point... Greetings, Andres Freund