Re: Typo: llvm*.cpp files identified as llvm*.c - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: Typo: llvm*.cpp files identified as llvm*.c
Date
Msg-id CAEepm=1rw0CCCufcD2gDxY18sHMh8O1FGPGgAEBrB5Gak9wojQ@mail.gmail.com
Whole thread Raw
In response to Re: Typo: llvm*.cpp files identified as llvm*.c  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
Responses Re: Typo: llvm*.cpp files identified as llvm*.c  (Michael Paquier <michael@paquier.xyz>)
RE: Typo: llvm*.cpp files identified as llvm*.c  ("Moon, Insung" <Moon_Insung_i3@lab.ntt.co.jp>)
Re: Typo: llvm*.cpp files identified as llvm*.c  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
List pgsql-hackers
On Wed, Jan 23, 2019 at 1:16 PM Amit Langote
<Langote_Amit_f8@lab.ntt.co.jp> wrote:
> On 2019/01/23 4:51, Andres Freund wrote:
> > On 2019-01-22 13:43:32 +0900, Amit Langote wrote:
> >> Attached find a patch to fix $subject.
> >
> > Thanks, pushed to master and 11.
>
> Thank you.

It's not only the ending that's wrong.  Here are some more source
files whose IDENTIFICATION heading doesn't exactly match their path:

$ git grep -A 1 IDENTIFICATION | grep -v IDENTIFICATION | grep -v --
-- | sed 's/-[^a-z][^a-z]*/ /' | awk '{ if ($1 != $2) print; }'
doc/src/sgml/lobj.sgml src/test/examples/testlo.c
src/backend/catalog/pg_publication.c pg_publication.c
src/backend/commands/publicationcmds.c publicationcmds.c
src/backend/commands/subscriptioncmds.c subscriptioncmds.c
src/backend/jit/llvm/llvmjit_inline.cpp
src/backend/lib/llvmjit/llvmjit_inline.cpp
src/backend/jit/llvm/llvmjit_wrap.cpp src/backend/lib/llvm/llvmjit_wrap.cpp
src/backend/optimizer/util/appendinfo.c src/backend/optimizer/path/appendinfo.c
src/backend/optimizer/util/inherit.c src/backend/optimizer/path/inherit.c
src/backend/replication/logical/logicalfuncs.c
src/backend/replication/logicalfuncs.c
src/backend/replication/logical/reorderbuffer.c
src/backend/replication/reorderbuffer.c
src/backend/replication/logical/snapbuild.c src/backend/replication/snapbuild.c
src/backend/replication/pgoutput/Makefile src/backend/replication/pgoutput
src/backend/utils/adt/version.c
src/include/replication/pgoutput.h pgoutput.h

This could be really confusing for erm, future people reading a dot
matrix print-out of the source code?

-- 
Thomas Munro
http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: COPY FROM WHEN condition
Next
From: Nikita Glukhov
Date:
Subject: Re: jsonpath