diff options
| author | David Rowley | 2020-11-24 21:52:50 +0000 |
|---|---|---|
| committer | David Rowley | 2020-11-24 21:52:50 +0000 |
| commit | b0727ae99b7056ca88e9708c014a86f6e611d938 (patch) | |
| tree | e9ef75375ca08f70dadae97944e72983b6b025ed /src/include/utils | |
| parent | ec05bafdbbf474bf0a1416772da31f9f1f27fa1e (diff) | |
Tidy up definitions of pg_attribute_hot and pg_attribute_cold
1fa22a43a was a quick fix for portability problem I introduced in
697e1d02f. 1fa22a43a adds a few more cases to the preprocessor logic than
I'd have liked. Andres Freund and Dagfinn Ilmari Mannsåker suggested a
better way to do this.
In passing, also adjust the only current usage of these macros so that the
macro comes before the function's return type in the declaration of the
function. This now matches what the definition of the function does.
Discussion: https://postgr.es/m/20200625163553.lt6wocbjhklp5pl4@alap3.anarazel.de
Discussion: https://postgr.es/m/87pn43bmok.fsf@wibble.ilmari.org
Diffstat (limited to 'src/include/utils')
| -rw-r--r-- | src/include/utils/elog.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/include/utils/elog.h b/src/include/utils/elog.h index cb7f8c30c30..e8f04a16916 100644 --- a/src/include/utils/elog.h +++ b/src/include/utils/elog.h @@ -160,7 +160,7 @@ extern bool message_level_is_interesting(int elevel); extern bool errstart(int elevel, const char *domain); -extern bool pg_attribute_cold errstart_cold(int elevel, const char *domain); +extern pg_attribute_cold bool errstart_cold(int elevel, const char *domain); extern void errfinish(const char *filename, int lineno, const char *funcname); extern int errcode(int sqlerrcode); |
