Andres Freund <andres@anarazel.de> writes:
> On 2016-01-18 12:52:06 +0100, Shulgin, Oleksandr wrote:
>> I've noticed that in src/backend/replication/logical/reorderbuffer.c, the
>> IDENTIFICATION comment line is incorrect:
>>
>> * IDENTIFICATION
>> - * src/backend/catalog/dropcmds.c
>> - * src/backend/replication/logicalfuncs.c
>> - * src/backend/replication/reorderbuffer.c
>> - * src/backend/replication/snapbuild.c
>> - * src/backend/storage/ipc/dsm.c
>> - * src/backend/utils/cache/relfilenode.c
>> - * src/backend/utils/cache/evtcache.c
> How about we simply drop them instead alltogether? This isn't exactly a
> seldomly made mistake, and they seem to actually contribute very little
> value?
Personally I think they're of some value. Particularly with stuff like
Makefiles, which are otherwise confusingly similar.
regards, tom lane