Skip to content

clang >= 3.7 incompatibility #119

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
konserw opened this issue Sep 3, 2016 · 4 comments
Closed

clang >= 3.7 incompatibility #119

konserw opened this issue Sep 3, 2016 · 4 comments

Comments

@konserw
Copy link
Contributor

konserw commented Sep 3, 2016

Hello,
I've just noticed 2 problems when compiling with clang version 3.8.1 on GNU/Linux (most probably problem exists for clang >= 3.7 both on Linux and MAC)

  • WireProtocols test are generating such warning:
    expression with side effects will be evaluated despite being used as an operand to 'typeid'
    on lines where EXPECT_PTRTYPE macro is used
  • SocketServer tests are failing with some exception:
    The following tests FAILED: 21 - SocketServerTest.exitsOnFirstConnectionClosed (OTHER_FAULT) 22 - SocketServerTest.moreThanOneClientCanConnect (OTHER_FAULT) 23 - SocketServerTest.receiveAndSendsSingleLineMassages (OTHER_FAULT)

Those 2 issues may or may not be connected with each other. Everything works perfectly fine using gcc 6.1.1 and of course on our CI builds.

I'm not sure how to fix that, but please let me know if I can help with this ;)

@konserw
Copy link
Contributor Author

konserw commented Sep 17, 2016

Socketserver tests appear to be fixed by #120 - not sure how. But warnings are still to be investigated in my opinion.
BTW. I'm very curious how #120 fixed those tests ;)

@paoloambrosio
Copy link
Member

@konserw what Linux distro are you using? I can't reproduce it on Ubuntu with clang++ 3.8.0-2ubuntu4

@konserw
Copy link
Contributor Author

konserw commented Mar 25, 2017

Arch Linux (Antegros)
clang --version
clang version 3.9.1 (tags/RELEASE_391/final)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

But the warning is also visible on travis OSX jobs, for instance: https://travis-ci.org/cucumber/cucumber-cpp/jobs/214419938

@paoloambrosio
Copy link
Member

I must have done something wrong the first time. Now I can see the warning :-)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants