On Sat, Sep 14, 2019 at 03:03:57PM +0900, Michael Paquier wrote: > On Sat, Sep 14, 2019 at 11:18:37AM +0530, Amit Kapila wrote: >> Won't 'Lateral' clause be helpful here as the patch contains it in one >> of its tests? > > Ah true, I forgot that.
If we are redesigning the interface, here are two extra thoughts which may be worth considering: 1) If the function returns multiple columns, could it make sense to separate infomask and infomask2? This would then give 3 columns: - The raw flags for infomask. - The three combined flags for infomask. - The flags for infomask2. 2) Could it make sense to have a separate function for infomask2? I'd rather keep everything in a single function, still as we are discussing the matter.. -- Michael
signature.asc
Description: PGP signature
