ne 2. 10. 2022 v 22:52 odesílatel Daniel Gustafsson <daniel@yesql.se> napsal:
> On 2 Oct 2022, at 18:04, Andres Freund <andres@anarazel.de> wrote: > On 2022-10-02 00:19:59 -0700, Andres Freund wrote: >> On 2022-10-01 23:56:59 -0700, Andres Freund wrote: >>> On 2022-09-12 09:58:37 +0200, Daniel Gustafsson wrote: >>>> Correct, fixed in the attached. >>> >>> Updated patch adding meson compatibility attached. >> >> Err, forgot to amend one hunk :( > > That fixed it on all platforms but windows, due to copy-pasto. I really should > have stopped earlier yesterday...
Thanks for updating the patch!
The parser in the original submission was -1'd by me, and the current version proposed as an alternative. This was subsequently -1'd as well but no updated patch with a rewritten parser has been posted. So this is now stalled again.
You started rewriting it, but you didn't finish it.
Unfortunately, there is not a clean opinion on using bison's parser for this purpose. I understand that the complexity of this language is too low, so the benefit of using bison's gramatic is low too. Personally, I have not any problem using bison for this purpose. For this case, I think we compare two similarly long ways, but unfortunately, customers that have a problem with long command lines still have this problem.
Can we go forward? Daniel is strongly against handwritten parser. Is there somebody strongly against bison's based parser? There is not any other way.
I am able to complete Daniel's patch, if there will not be objections.
Regards
Pavel
Having been around in 12 commitfests without a committer feeling confident about pushing this I plan to mark it returned with feedback, and if a new parser materializes itc can be readded instead of being dragged along.
> c.h (and postgres.h, postgres_fe.h) shouldn't be included in headers. > > This is a common enough mistake that I'm wondering if we could automate > warning about it somehow.
Maybe we can add a simple git grep invocation in the CompilerWarnings CI job to catch this in the CFBot? If something like the below sketch matches then we can throw an error. (only for illustration, all three files needs to checked).
