Re: Enhanced error message to include hint messages for redundant options error

> On 6 Jul 2021, at 17:08, vignesh C <vignesh21@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/