NotificationsYou must be signed in to change notification settings
Fork28
Star153

Commitf2717c7

committed

Improve unreachability recognition in elog() macro.

Some experimentation with an older version of gcc showed that it is ableto determine whether "if (elevel_ >= ERROR)" is compile-time constantif elevel_ is declared "const", but otherwise not so much. We hadaccounted for that in ereport() but were too miserly with braces tomake it so in elog(). I don't know how many currently-interestingcompilers have the same quirk, but in case it will save some codespace, let's make sure that elog() is on the same footing as ereport()for this purpose.Back-patch to 9.3 where we introduced pg_unreachable() calls intoelog/ereport.

1 parentddc8893 commitf2717c7Copy full SHA for f2717c7

File tree

1 file changed

-5

lines changed

src/include/utils
- elog.h

1 file changed

-5

lines changed

`‎src/include/utils/elog.h‎`

Lines changed: 6 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -206,12 +206,13 @@ extern intgetinternalerrposition(void);`
`206`	`206`	`#else/* !HAVE__BUILTIN_CONSTANT_P */`
`207`	`207`	`#defineelog(elevel, ...) \`
`208`	`208`	`do { \`
`209`		`-intelevel_; \`
`210`	`209`	`elog_start(__FILE__, __LINE__, PG_FUNCNAME_MACRO); \`
`211`		`-elevel_ = (elevel); \`
`212`		`-elog_finish(elevel_, __VA_ARGS__); \`
`213`		`-if (elevel_ >= ERROR) \`
`214`		`-pg_unreachable(); \`
	`210`	`+{ \`
	`211`	`+const int elevel_ = (elevel); \`
	`212`	`+elog_finish(elevel_, __VA_ARGS__); \`
	`213`	`+if (elevel_ >= ERROR) \`
	`214`	`+pg_unreachable(); \`
	`215`	`+} \`
`215`	`216`	`} while(0)`
`216`	`217`	`#endif/* HAVE__BUILTIN_CONSTANT_P */`
`217`	`218`	`#else/* !HAVE__VA_ARGS */`

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!

Commitf2717c7

File tree

1 file changed

1 file changed

`‎src/include/utils/elog.h‎`

0 commit comments