diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2019-05-31 11:45:33 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2019-05-31 11:45:33 -0400 |
| commit | 7640f9312321ceba2af61f7017da7e64f0f7b667 (patch) | |
| tree | bd42ca762cdd5769c23f06c7888bd0f72e351402 /src/pl/plpython/plpython.h | |
| parent | 9e58705a7f9728f10b2e0f0c9adc6e6ecff7486e (diff) | |
| download | postgresql-7640f9312321ceba2af61f7017da7e64f0f7b667.tar.gz postgresql-7640f9312321ceba2af61f7017da7e64f0f7b667.zip | |
Fix assorted header files that failed to compile standalone.
We have a longstanding project convention that all .h files should
be includable with no prerequisites other than postgres.h. This is
tested/relied-on by cpluspluscheck. However, cpluspluscheck has not
historically been applied to most headers outside the src/include
tree, with the predictable consequence that some of them don't work.
Fix that, usually by adding missing #include dependencies.
The change in printf_hack.h might require some explanation: without
it, my C++ compiler whines that the function is unused. There's
not so many call sites that "inline" is going to cost much, and
besides all the callers are in test code that we really don't care
about the size of.
There's no actual bugs being fixed here, so I see no need to back-patch.
Discussion: https://postgr.es/m/b517ec3918d645eb950505eac8dd434e@gaz-is.ru
Diffstat (limited to 'src/pl/plpython/plpython.h')
| -rw-r--r-- | src/pl/plpython/plpython.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/pl/plpython/plpython.h b/src/pl/plpython/plpython.h index 1a9463a1e31..3a1f0d56d7c 100644 --- a/src/pl/plpython/plpython.h +++ b/src/pl/plpython/plpython.h @@ -14,7 +14,8 @@ /* * Include order should be: postgres.h, other postgres headers, plpython.h, - * other plpython headers + * other plpython headers. (In practice, other plpython headers will also + * include this file, so that they can compile standalone.) */ #ifndef POSTGRES_H #error postgres.h must be included before plpython.h |