On Sat, Jul 05, 2014 at 09:29:31PM +0200, Jakub Jelinek wrote: > On Sat, Jul 05, 2014 at 09:25:50PM +0200, Gerald Pfeifer wrote: > > On Sat, 5 Jul 2014, pins...@gmail.com wrote: > > > This patch is fine but c++ allows class and struct be used > > > interchangeable. If there is a compilers which does not it is broken > > > and should be report to them. Yes that means clang is broken. > > > > Clang does allow for it (it actually is the stage 1 compiler on > > FreeBSD 10, the platform I used for my tests); it just warns about > > it. About 400 times. > > That doesn't change anything that the warning is very much broken. > In C++ struct is simply a class with default public:, class with a default > private:, when you are just forward declaring it, whether it defaults to > public: or private: doesn't matter at all, therefore the warning just > enforces some weirdo clang coding style.
I thought the warning was to help people who care about MSVC and its utterly broken mangling, but I also agree its a pretty silly warning that I wouldn't mind turning off. Trev > > Jakub
