forked fromglynos/cpp-netlib
- Notifications
You must be signed in to change notification settings - Fork425
0.12-devel to be C++11-only#569
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
glynos merged 36 commits intocpp-netlib:0.12-develfromdeanberris:0.11-devel-cpp11-upgradeNov 18, 2015
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes fromall commits
Commits
Show all changes
36 commits Select commitHold shift + click to select a range
a370594
Run clang-tidy with all checks.
deanberris44bd36b
Manual changes to update style and fix breakages
deanberris5f6e9ee
Changes to make things build
deanberris7b88a41
Reintroduce missing code for parsing query maps
deanberris6349c11
clang-tidy modernize-.* all the things
deanberris6d8879c
Upgrade travis config
deanberris65a0e6e
Fix typo in URL
deanberrisfeec209
Fix another typo on the URL.
deanberris4847865
Make travis wait for a Boost build to finish
deanberrise19af74
Change level of travis_wait call; chmod +x .sh files
deanberrisabe68b4
Only support Boost 1.59 for now.
deanberrisb10dfbb
Only produce shared+multithreaded debug & release boost libs
deanberris8cb4c16
Looks like we need static libs too
deanberris198dd07
Integrate travis; Use Boost 1.57 at least; Always use shared libs fro…
deanberrisf6c7ee6
Use the dynamic version of Boost.Test always.
deanberrisde7c710
Use v4 again instead of just address
deanberris39a2705
v4() for real this time
deanberris2e3340f
Change test to fetch different targets
deanberris7e0f93e
Simplify http test
deanberris552401b
Use ninja-build
deanberris19beafb
Update cmake
deanberris99f6168
Use updated cmake to support ninja-build
deanberris937d589
Travis cannot handle ninja builds yet, probably to lack of resources …
deanberrisd5ec24e
j4 is too much. :(
deanberrisd661fcc
Check with the sanitizers
deanberris5751338
Honor the CMAKE_CXX_FLAGS environment variable
deanberris511dd04
Print latest log properly
deanberris9fbc300
Fix some issues with the (deprecated) synchronous client implementati…
deanberris2f2239c
Only use libc++ if in OS X
deanberris4be4eae
Issue identified with memory sanitizer
deanberris600daff
Fix unsafe usage of boost::as_literal, caught by memory sanitiser
deanberris03870ca
Force use of size_t in distance calculation
deanberris9bad07c
Removing noexcept from defaulted move constructor
deanberris2f2c021
Track origins for memsan runs
deanberris7f70dac
Exclude msan builds from g++
deanberris8a8279a
Do not use msan yet; Boost seems to be not msan-clean
deanberrisFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
1 change: 1 addition & 0 deletions.gitignore
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -10,3 +10,4 @@ libs/mime/test/mime-roundtrip | ||
*.a | ||
_build | ||
/.project | ||
build/ |
106 changes: 56 additions & 50 deletions.travis.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,55 +1,61 @@ | ||
sudo: false | ||
language: cpp | ||
compiler: | ||
- g++ | ||
- clang | ||
env: | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="ON" CMAKE_BUILD_TYPE="Release" ENABLE_HTTPS="ON" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="ON" CMAKE_BUILD_TYPE="Release" ENABLE_HTTPS="OFF" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="ON" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="ON" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="OFF" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Release" ENABLE_HTTPS="ON" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Release" ENABLE_HTTPS="OFF" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="OFF" | ||
# Support the sanitizers in clang only | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" CMAKE_CXX_FLAGS="-fsanitize=thread" | ||
- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" CMAKE_CXX_FLAGS="-fsanitize=address" | ||
# TODO(deanberris): It seems Boost is not msan-clean yet; report bugs and maybe fix? | ||
#- BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" CMAKE_CXX_FLAGS="-fsanitize=memory -fsanitize-memory-track-origins=2" | ||
matrix: | ||
exclude: | ||
- compiler: g++ | ||
env: BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" CMAKE_CXX_FLAGS="-fsanitize=thread" | ||
- compiler: g++ | ||
env: BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" CMAKE_CXX_FLAGS="-fsanitize=address" | ||
# TODO(deanberris): It seems Boost is not msan-clean yet; report bugs and maybe fix? | ||
# - compiler: g++ | ||
# env: BOOST_VER=1.59.0 BUILD_SHARED_LIBS="OFF" CMAKE_BUILD_TYPE="Debug" ENABLE_HTTPS="ON" CMAKE_CXX_FLAGS="-fsanitize=memory -fsanitize-memory-track-origins=2" | ||
install: | ||
- mkdir -p ${HOME}/bin | ||
- if [ "${CC}" = "gcc" ]; then export TOOLSET="gcc"; ln -s `which g++-4.8` ${HOME}/bin/g++; | ||
ln -s `which gcc-4.8` ${HOME}/bin/gcc; fi | ||
- if [ "${CC}" = "clang" ]; then export TOOLSET="clang"; ln -s `which clang-3.6` ${HOME}/bin/clang; | ||
ln -s `which clang++-3.6` ${HOME}/bin/clang++; fi | ||
- export BOOST_VERSION=${BOOST_VER//./_} | ||
- export PATH=${HOME}/bin:${PATH} | ||
- travis_wait ./install-boost.sh | ||
- export BOOST_ROOT=${HOME}/${CC}-boost_${BOOST_VER//./_} | ||
- "${CXX} --version" | ||
cache: | ||
directories: | ||
- "${HOME}/${CC}-boost_${BOOST_VER//./_}" | ||
script: | ||
- pwd | ||
- sh -x build.sh | ||
after_failure: | ||
- cat build/Testing/Temporary/LastTest.log | ||
addons: | ||
apt: | ||
sources: | ||
- ubuntu-toolchain-r-test | ||
- llvm-toolchain-precise-3.6 | ||
- kalakris-cmake | ||
packages: | ||
- gcc-4.8 | ||
- g++-4.8 | ||
- clang-3.6 | ||
- cmake | ||
notifications: | ||
slack: | ||
secure: Y7lLjqZ83+b/jaJ5+EKwvgCDeERi4bVbDn9tLp8sieTdu+ENsPI+JmLYSXZXPpe7JrItrXW6uJJXN2wG1h7au4mpVVTghd31HBzuzrqVxDphWPhp16NYzvbAgQQRBXvFVvfSdW/Kb/n2fX6xDApY0t6vNREb/GKg0GyzESb4ZjU= |
2 changes: 1 addition & 1 deletion.ycm_extra_conf.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -17,7 +17,7 @@ | ||
'-Wall', | ||
'-Wextra', | ||
'-Werror', | ||
'-std=c++11', | ||
'-isystem', | ||
'.', | ||
'-isystem', | ||
28 changes: 16 additions & 12 deletionsCMakeLists.txt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
42 changes: 19 additions & 23 deletionsboost/mime.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
6 changes: 3 additions & 3 deletionsboost/network.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
4 changes: 2 additions & 2 deletionsboost/network/constants.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
11 changes: 7 additions & 4 deletionsboost/network/detail/directive_base.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
6 changes: 2 additions & 4 deletionsboost/network/detail/wrapper_base.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Oops, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.