Horton

Re: Remove useless arguments in ReadCheckpointRecord().

Fujii Masao <masao.fujii@oss.nttdata.com>

View thread View on postgr.es
From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Cc: bharath.rupireddyforpostgres@gmail.com, pgsql-hackers@postgresql.org
Date: 2022-07-25T02:30:30Z
Lists: pgsql-hackers 

On 2022/07/22 17:31, Kyotaro Horiguchi wrote:
> I believed that it is recommended to move to the style not having the
> outmost parens.  That style has been introduced by e3a87b4991.

I read the commit log, but I'm not sure yet if it's really recommended to remove extra parens even from the existing calls to errmsg(). Removing extra parens can interfere with back-patching of the changes around those errmsg(), can't it?

Anyway, at first I pushed the 0001 patch that removes useless arguments in ReadCheckpointRecord().

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

Commits

  1. Remove useless arguments in ReadCheckpointRecord().

  2. Remove unnecessary call to ReadCheckpointRecord().