Hi, 

On April 10, 2026 3:57:56 AM EDT, Thomas Munro <[email protected]> wrote:
>Nitpicking code review for commit b15c1513:
>
>+read_archive_wal_page(XLogDumpPrivate *privateInfo, XLogRecPtr targetPagePtr,
>+                     Size count, char *readBuff)
>
>I thought we agreed to stop using Size for new code?  size_t has been
>around since C89.

We really need to start documenting some of this stuff somewhere. Deciding 
something a few years ago, deep in a thread, won't actually help anyone but the 
participants (and maybe not even them) to know about it. 

I wonder if we should move the coding style section out of sgml into a 
top-level CODING_STYLE.md or something like that.

And then obviously add things like Size being deprecated. 

Greetings, 

Andres 


-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Reply via email to