Re: More CppAsString2() in psql's describe.c
Corey Huinker <corey.huinker@gmail.com>
From: Corey Huinker <corey.huinker@gmail.com>
To: Daniel Gustafsson <daniel@yesql.se>
Cc: Michael Paquier <michael@paquier.xyz>, Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2024-11-28T18:25:56Z
Lists: pgsql-hackers
> > > LGTM, I didn't scan for omissions but the ones in the patch look right to > me. > I sort of wish we had a shorter macro as CppAsString2() get's pretty > verbose > when used frequently like this. > I don't quite understand the etymology of the name (it's some variation on C++'s std::to_string, plus...something), but if I did, I'd probably find the name less icky. STR(), C_STR(), STRING(), and CSTRING() all seem to be available...
Commits
-
psql: Sprinkle more CppAsString2() in describe.c
- 18954ce7f69b 18.0 landed