mirror of
https://github.com/blender/blender
synced 2026-10-25 03:07:01 +03:00
344b741144
Clang on windows has always been generous with the amount of warnings
it emitted, no-one really had/took the time to take a closer look here
and we just suppressed [1] the warnings and moved on with our day.
However every-time a new clang version gets released more and more
warnings had to be suppressed, the tests for which flags are available
was reported to take 5+ minutes for some users, with more devs taking
clang for a spin, time to finally take that closer look!
Jonas dove in the docs and figured out that passing `-Wall` for clang-cl
doesn't do what you think it does and actually enables `-Weverything`[2]
which is the root cause for the abundance of warnings by clang on
windows.
This PR:
- Removes `-Wall` for clang on windows and replaces it with /W3 (which
will imply an _actual_ -Wall)
- Removes the large list of suppressions we had for clang on Windows
and synchronizes with the existing warning flags for Mac/Linux.
With this done there were about 200 warnings left in mostly windows
specific code (See PR for specifics) which this PR also fixes.
[1]
|
||
|---|---|---|
| .. | ||
| include/glog | ||
| src | ||
| AUTHORS | ||
| ChangeLog | ||
| CMakeLists.txt | ||
| COPYING | ||
| NEWS | ||
| README | ||
| README.blender | ||
This repository contains a C++ implementation of the Google logging module. Documentation for the implementation is in doc/. See INSTALL for (generic) installation instructions for C++: basically ./configure && make && make install