From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: remove ancient pre-dlopen dynloader code |
Date: | 2018-08-16 13:22:14 |
Message-ID: | 8022.1534425734@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andres Freund <andres(at)anarazel(dot)de> writes:
> This now generates a super nitpicky warning on at at least some linux +
> clang configurations. I use -Weverything plus a lot of -Wno-*, and this
> change added:
> dynloader.c:7:4: warning: ISO C requires a translation unit to contain at least one declaration [-Wempty-translation-unit]
We've been seeing that (or equivalents) on other platforms for years,
if not decades. I can't get too excited about it really.
The lazy man's way to get rid of it would be to put something like
"int bogus = 0;" in the empty dynloader.c files. Better would be
to not have the empty .c files at all, but I'm not sure how much
we'd have to contort the Makefiles to support that.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Dian Fay | 2018-08-16 13:36:29 | Re: docs: note ownership requirement for refreshing materialized views |
Previous Message | Andres Freund | 2018-08-16 13:00:30 | Re: C99 compliance for src/port/snprintf.c |