Warnings with g++

[goto40]

I just tried xtensor. Gread work!

However, I found some warnings when compiling with my g++ (gcc 9.3.0) for different warnings configurations (e.g. Weffc++, pedantic, ...).
E.g.

[build] /.../xtensor/xoperation.hpp:86:35: warning: extra ‘;’ [-Wpedantic]
[build]    86 |         DEFINE_COMPLEX_OVERLOAD(+);

Is there already some work going on for such issues (I searched the github issues and did not found anything)? Else we could use this ticket for PRs concerning warnings cleanups (and I could add some findings)?

Or is there a specific warning setting to be used/ignored for some reason?

(I am aware of -isystem, which could help me out in that case by ignoring those warnings.)

[JohanMabille]

Thanks for opening this issue!

There isn't any opened issue to track the warning at the moment. Sometimes we take a few days to remove as much warnings as we can, but we limit ourselves to the CI settings, so this issue very welcome!

I don't think we have some warnings that should be ignored, so let's use this issue to track all of them.

We can also add another configuration to the CI so it's easier for everyone to catch these warnings. Any PR to remove them is very welcome.