Re: warnings in unit tests

2021-06-08 Thread Dmitry V. Levin
On Tue, Jun 08, 2021 at 10:56:33AM +0200, Bruno Haible wrote: > Jim Meyering wrote: > > I can live without -Wmissing-prototypes in gnulib tests, but I still > > remember times where using that option exposed a real bug. > > -Wmissing-prototypes typically exposes real bugs when a program is

Re: warnings in unit tests

2021-06-08 Thread Paul Eggert
For what it's worth I'm more with Bruno on this. For the tests, the cost of these warnings outweighs the benefit. It'd be OK with me to disable the troublesome warnings globally for the tests subdirectory, using -Wno-missing-prototypes or whatever. For the -Wnull-dereference issue it may be

Re: warnings in unit tests

2021-06-08 Thread Bruno Haible
Jeffrey Walton wrote: > Apple's port of Clang enables missing prototypes by default. You will > get the warnings whether you use -Wmissing-prototypes or not. You now > have to actively disable the warning with -Wno-missing-prototypes. Thanks for the heads-up. From my perspective, we can treat

Re: warnings in unit tests

2021-06-08 Thread Bruno Haible
Jim Meyering wrote: > I can live without -Wmissing-prototypes in gnulib tests, but I still > remember times where using that option exposed a real bug. -Wmissing-prototypes typically exposes real bugs when a program is composed of several compilation units. Unit tests are typically a single

Re: [PATCH] mountlist: recognize fuse.portal as dummy file system

2021-06-08 Thread Kamil Dudka
On Tuesday, June 8, 2021 12:16:36 AM CEST Pádraig Brady wrote: > On 07/06/2021 13:43, Kamil Dudka wrote: > > This was originally proposed at: > > https://lists.gnu.org/archive/html/bug-gnulib/2021-02/msg00053.html > > > > As the full review might take some time, would it be possible to apply