Horton

Re: meson: Specify -Wformat as a common warning flag for extensions

Sutou Kouhei <kou@clear-code.com>

View thread View on postgr.es
From: Sutou Kouhei <kou@clear-code.com>
To: andres@anarazel.de
Cc: peter@eisentraut.org, tristan@neon.tech, michael@paquier.xyz, pgsql-hackers@postgresql.org
Date: 2024-04-08T01:01:17Z
Lists: pgsql-hackers

Attachments

Hi Andres,

Thanks for reviewing this!

In <20240407232635.fq4kc5556lahaoej@awork3.anarazel.de>
  "Re: meson: Specify -Wformat as a common warning flag for extensions" on Sun, 7 Apr 2024 16:26:35 -0700,
  Andres Freund <andres@anarazel.de> wrote:

> This seems like a fair amount of extra configure tests. Particularly because
> /W* isn't ever interesting for Makefile.global - they're msvc flags - because
> you can't use that with msvc.
> 
> I'm also doubtful that it's worth supporting warning_level=3/everything, you
> end up with a completely flood of warnings that way.

OK. I've removed "/W*" flags and warning_level==3/everything
cases.

How about the attached v5 patch?


Thanks,
-- 
kou

Commits

  1. meson: Restore implicit warning/debug/optimize flags for extensions