Skip to content
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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Run ThemisPP tests with C++14 and C++17 #572

Merged
merged 3 commits into from
Dec 24, 2019
Merged

Run ThemisPP tests with C++14 and C++17 #572

merged 3 commits into from
Dec 24, 2019

Commits on Dec 23, 2019

  1. Run ThemisPP tests with C++14 and C++17 

    Explicitly test compatibility with those standard levels. C++20 is
currently being finalized. It's somewhat supported in the compilers,
but not in the versions shipped with Ubuntu 16.04.
    @ilammy
    ilammy committed Dec 23, 2019
    Configuration menu
    Copy the full SHA
    d742d91 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into cpp-standards

    @ilammy
    ilammy committed Dec 23, 2019
    Configuration menu
    Copy the full SHA
    e0484b7 View commit details
    Browse the repository at this point in the history

Commits on Dec 24, 2019

  1. Run ThemisPP tests with both GCC and Clang 

    It turned out that behavior wrt attributes might be different between
GCC and Clang, so let's test both compilers now.

Clang that we have in the Docker image does not support "-std=c++17",
so we do not test with that. (Actually, GCC support the option, but
__cplusplus value suggests that it's still in C++14 mode.)
    @ilammy
    ilammy committed Dec 24, 2019
    Configuration menu
    Copy the full SHA
    7722157 View commit details
    Browse the repository at this point in the history