NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitff47d4b

committed

Work around stdbool problem in dfmgr.c.

Commit842cb9f refactored things so that dfmgr.c includes <dlfcn.h>,which before that had only been directly included in platform-specificstub files. It turns out that on macOS, <dlfcn.h> includes <stdbool.h>,and that causes problems on platforms where _Bool is not char-sized ...which happens to include the PPC versions of macOS. Work around itmuch as we have in plperl.h, by #undef'ing bool after including theproblematic file, but only if we're not using stdbool-style booleans.Discussion:https://postgr.es/m/E1fxqjl-0003YS-NS@gemulon.postgresql.org

1 parented0cdf0 commitff47d4bCopy full SHA for ff47d4b

File tree

1 file changed

+10

-0

lines changed

src/backend/utils/fmgr
- dfmgr.c

1 file changed

+10

-0

lines changed

`‎src/backend/utils/fmgr/dfmgr.c`

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,17 @@`
`18`	`18`
`19`	`19`	`#ifdefHAVE_DLOPEN`
`20`	`20`	`#include<dlfcn.h>`
	`21`	`+`
	`22`	`+/*`
	`23`	`+ * On macOS, <dlfcn.h> insists on including <stdbool.h>. If we're not`
	`24`	`+ * using stdbool, undef bool to undo the damage.`
	`25`	`+ */`
	`26`	`+#ifndefUSE_STDBOOL`
	`27`	`+#ifdefbool`
	`28`	`+#undef bool`
	`29`	`+#endif`
`21`	`30`	`#endif`
	`31`	`+#endif/* HAVE_DLOPEN */`
`22`	`32`
`23`	`33`	`#include"fmgr.h"`
`24`	`34`	`#include"lib/stringinfo.h"`

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!

Commitff47d4b

File tree

1 file changed

1 file changed

`‎src/backend/utils/fmgr/dfmgr.c`

0 commit comments