A number of fixes for compiler warnings #20
Conversation
... for the purposes of this project and makes testing on older platforms easier. In addition travis-ci has by default only cmake 2.8.
... which allows to use "make test" and/or "ctest -V"
... on clang-3.7
... which allows to use peglib.h in a C++11 only environment.
VS2010 is not capable of compiling source code with chars beyond the "current code page" reliably.
|
@hvellyr, thanks for the valuable contribution! |
|
Hi, I can't imagine that this outage will stay permanently. Wether llvm.org opens their apt download again, or that service is hosted somewhere else, I don't know. It's basically all a problem about travis using too old ubuntu versions (default is 12.02 I think), where clang is still a 3.4 (or so) with limited c++11 support. I wanted to experiment with a different base version (trusty seems to be available in travis, i.e. 14.04), but didn't had the time to do so. Alternatively one could turn off the container support (i.e. Since a lot of projects are affected by this outage I'm pretty someone will come up with some solution. For now best is probably to disable clang 😞 (BTW. The "official" issue with the travis whitelist is here travis-ci/apt-source-safelist#156) |
|
Thanks for the info. I disabled the clang section for now, and added the travis-ci build status badge on README. It's great to see the green 'passing' badge!! |
Since I'm considering using this code in an environment which has much stricter warnings level set, I found quite a number of places to improve the code. To achieve this I propose some CMake enhancement plus automatic testing on travis-ci.org. (Unluckily llvm.org decided to drop their apt download which makes building with clang on linux boxes on travis-ci.org fail currently).