On Sat, Sep 17, 2022 at 10:00 PM Peter Geoghegan wrote:
> I actually noticed that clang-tidy missed this one myself, shortly
> after commit. I still haven't figured out why, but it seems like it
> might be a bug in the readability-named-parameter check. Could also
> have something to do with my cl
Harmonize missed reorderbuffer parameter names.
The function ReorderBufferCommitChild() was overlooked by initial work
from commit 035ce1fe.
Author: Peter Geoghegan
Discussion:
https://postgr.es/m/cah2-wzkhzfesnro+vagqyezuzc33dw09bdzbvmw896sa22c...@mail.gmail.com
Branch
--
master
Details
Peter Geoghegan writes:
> I see what happened here: while the
> "readability-inconsistent-declaration-parameter-name" check works
> equally well with extern functions and functions with local linkage,
> the same cannot be said for the "readability-named-parameter" check.
> The latter only seems to
On Sun, Sep 18, 2022 at 2:01 PM Tom Lane wrote:
> That's kind of annoying --- seems to put a serious crimp in any plans
> to check this mechanically.
I don't see why it should make a huge difference. Granted we can't
really rely on the "readability-named-parameter" check in the way we'd
hoped, bu
On Sun, Sep 18, 2022 at 2:14 PM Peter Geoghegan wrote:
> One further caveat here is that I seem to need to set "IgnoreMacros:
> false" to get perfect results for the "inconsistent" check when the C
> preprocessor is involved, as it often is (e.g., with TransactionId
> params).
Actually that expla
Fix typos referring to PGPROC
Japin Li
Reviewed by Kyotaro Horiguchi
Discussion:
https://www.postgresql.org/message-id/meyp282mb1669459813b36fb5eaa38434b6...@meyp282mb1669.ausp282.prod.outlook.com
Branch
--
master
Details
---
https://git.postgresql.org/pg/commitdiff/08f8af983a8bd8ef78e
