On Wed, Sep 04, 2024 at 09:17:15AM +1000, Peter Smith wrote: > I have merged the patches 0001-0004 as suggested. Please see v11 attachments.
Thanks. It took me some time to go through the whole tree for more inconsistencies. In 0001, search_path was missing quotes in vacuumlo.c and oid2name.c. Not the most critical tools ever, still fixed these. CheckRequiredParameterValues() has two "wal_level=minimal". Shouldn't separate quotes be used for the GUC name and its value to be more consistent with the rest? There are also two "full_page_writes=off" in xlog.c. Point mentioned at [1] by Daniel on v6, changed as they are on HEAD by 17974ec25946. In 0004, there are a couple of changes where this does not represent a gain for translators, and it was even made worse. For example huge_page_size updated for sysv but not WIN32, leading to two messages. The changes in postgres.c, predicate.c, syncrep.c and variable.c don't show a gain. The changes in dfmgr.c should have a translator note, I think, because it becomes unclear what these messages are about. By the way, I don't get why we use "/*- translator:" in some places while we document to use "/* translator:" in the NLS section of the docs. One pattern is much more used than the other, guess which one. 0001 and 0004 have been applied with these tweaks. I am still not sure about the changes for DateStyle and IntervalStyle in 0002 and 0003. Perhaps others have an opinion that could drive to a consensus. [1]: https://www.postgresql.org/message-id/2162891.1715890...@sss.pgh.pa.us -- Michael
signature.asc
Description: PGP signature