Re: [HACKERS] [PATCH] Suppress Clang 3.9 warnings

Поиск
Список
Период
Сортировка
От Noah Misch
Тема Re: [HACKERS] [PATCH] Suppress Clang 3.9 warnings
Дата
Msg-id 20170315160940.GA1961015@tornado.leadboat.com
обсуждение исходный текст
Ответ на Re: [HACKERS] [PATCH] Suppress Clang 3.9 warnings  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [HACKERS] [PATCH] Suppress Clang 3.9 warnings  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Wed, Mar 15, 2017 at 10:57:15AM -0400, Tom Lane wrote:
> Noah Misch <noah@leadboat.com> writes:
> > On Mon, Mar 13, 2017 at 06:35:53PM +0300, Aleksander Alekseev wrote:
> >> + * Unfortunately in case of strlcat and strlcpy we can't trust tests
> >> + * executed by Autotools if Clang > 3.6 is used.
> 
> > This is wrong on platforms that do have strlcpy() in libc.
> 
> Didn't you submit a patch to upstream autoconf awhile ago to fix the
> AC_CHECK_DECLS test for this?

Yes.

> Seems like the correct solution is to
> absorb that fix, either by updating to a newer autoconf release or by
> carrying our own version of AC_CHECK_DECLS until they come out with one.

As you mention upthread, that Autoconf commit is still newer than every
Autoconf release.  (Last release was 58 months ago.)  Altering configure.ac to
work around the bug would be reasonable, but it feels heavy relative to the
benefit of suppressing some warnings.



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Dilip Kumar
Дата:
Сообщение: Re: [HACKERS] Parallel Bitmap scans a bit broken
Следующее
От: Stephen Frost
Дата:
Сообщение: [HACKERS] Re: new set of psql patches for loading (saving) data from (to)text, binary files