> On 6 Jul 2021, at 17:08, vignesh C <vignes...@gmail.com> wrote: > The patch was not applying on the head because of the recent commit > "8aafb02616753f5c6c90bbc567636b73c0cbb9d4", attached patch which is > rebased on HEAD.
I sort of like the visual cue of seeing ereport(ERROR .. since it makes it clear it will break execution then and there, this will require a lookup for anyone who don't know the function by heart. That being said, reducing duplicated boilerplate has clear value and this reduce the risk of introducing strings which are complicated to translate. On the whole I think this is a net win, and the patch looks pretty good. - DefElem *defel = (DefElem *) lfirst(option); + defel = (DefElem *) lfirst(option); Any particular reason to include this in the patch? -- Daniel Gustafsson https://vmware.com/
