windows: new -Wreturn-type warnings after #19203 #21355
Description
We now get lots of compile time output like:
In file included from test/fuzz/addition_overflow.cpp:7:
./test/fuzz/util.h: In member function 'virtual SOCKET FuzzedSock::Get() const':
./test/fuzz/util.h:563:5: warning: no return statement in function returning non-void [-Wreturn-type]
563 | }
| ^
./test/fuzz/util.h: In member function 'virtual SOCKET FuzzedSock::Release()':
./test/fuzz/util.h:568:5: warning: no return statement in function returning non-void [-Wreturn-type]
568 | }
| ^
...anywhere test/fuzz/util.h is included. Would be great to not have build logs filled with this spam.
This was all present in the CI. i.e https://cirrus-ci.com/task/5500626558779392?command=ci#L4283.