NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit23c1232

committed

plpython: Avoid the need to redefine *printf macros

Until now we undefined and then redefined a lot of *printf macros due toworries about conflicts with Python.h macro definitions. Current Python.hdoesn't define any *printf macros, and older versions just defined snprintf,vsnprintf, guarded by #if defined(MS_WIN32) && !defined(HAVE_SNPRINTF).Thus we can replace the undefine/define section with a single #define HAVE_SNPRINTF 1Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>Discussion:https://postgr.es/m/20230124165814.2njc7gnvubn2amh6@awork3.anarazel.de

1 parent3b4ac33 commit23c1232Copy full SHA for 23c1232

File tree

1 file changed

-45

lines changed

src/pl/plpython
- plpython.h

1 file changed

-45

lines changed

`‎src/pl/plpython/plpython.h‎`

Lines changed: 3 additions & 45 deletions

Original file line number	Diff line number	Diff line change
`@@ -30,17 +30,10 @@`
`30`	`30`	`#undef _XOPEN_SOURCE`
`31`	`31`
`32`	`32`	`/*`
`33`		`- Sometimes python carefully scribbles on our printf macros.`
`34`		`- *So we undefine them here and redefine them after it's done its dirty deed.`
	`33`	`+ *Python versions <= 3.8 otherwise define a replacement, causing macro`
	`34`	`+ *redefinition warnings.`
`35`	`35`	`*/`
`36`		`-#undef vsnprintf`
`37`		`-#undef snprintf`
`38`		`-#undef vsprintf`
`39`		`-#undef sprintf`
`40`		`-#undef vfprintf`
`41`		`-#undef fprintf`
`42`		`-#undef vprintf`
`43`		`-#undef printf`
	`36`	`+#define HAVE_SNPRINTF 1`
`44`	`37`
`45`	`38`	`#if defined(_MSC_VER) && defined(_DEBUG)`
`46`	`39`	`/* Python uses #pragma to bring in a non-default libpython on VC++ if`
`@@ -63,41 +56,6 @@`
`63`	`56`	`#undef TEXTDOMAIN`
`64`	`57`	`#define TEXTDOMAIN PG_TEXTDOMAIN("plpython")`
`65`	`58`
`66`		`-/* put back our printf macros ... this must match src/include/port.h /`
`67`		`-#ifdef vsnprintf`
`68`		`-#undef vsnprintf`
`69`		`-#endif`
`70`		`-#ifdef snprintf`
`71`		`-#undef snprintf`
`72`		`-#endif`
`73`		`-#ifdef vsprintf`
`74`		`-#undef vsprintf`
`75`		`-#endif`
`76`		`-#ifdef sprintf`
`77`		`-#undef sprintf`
`78`		`-#endif`
`79`		`-#ifdef vfprintf`
`80`		`-#undef vfprintf`
`81`		`-#endif`
`82`		`-#ifdef fprintf`
`83`		`-#undef fprintf`
`84`		`-#endif`
`85`		`-#ifdef vprintf`
`86`		`-#undef vprintf`
`87`		`-#endif`
`88`		`-#ifdef printf`
`89`		`-#undef printf`
`90`		`-#endif`
`91`		`-`
`92`		`-#define vsnprintfpg_vsnprintf`
`93`		`-#define snprintfpg_snprintf`
`94`		`-#define vsprintfpg_vsprintf`
`95`		`-#define sprintfpg_sprintf`
`96`		`-#define vfprintfpg_vfprintf`
`97`		`-#define fprintfpg_fprintf`
`98`		`-#define vprintfpg_vprintf`
`99`		`-#define printf(...)pg_printf(__VA_ARGS__)`
`100`		`-`
`101`	`59`	`/*`
`102`	`60`	`* Used throughout, so it's easier to just include it everywhere.`
`103`	`61`	`*/`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit23c1232

File tree

1 file changed

1 file changed

`‎src/pl/plpython/plpython.h‎`

0 commit comments