Skip to content

Fix #14850 (Compilation fails on oraclelinux:8 (g++ 8.5.0 released in 2021)) #8654

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

Open
danmar wants to merge 7 commits into
base: main
Choose a base branch
from
+24 −6
Open

Fix #14850 (Compilation fails on oraclelinux:8 (g++ 8.5.0 released in 2021)) #8654

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
34bd599
Fix #14850 (Compilation fails on oraclelinux:8 (g++ 8.5.0 released in…
danmar Jun 16, 2026
bb014c5
format
danmar Jun 16, 2026
0850bef
cleanup
danmar Jun 16, 2026
50fec8e
rely on gcc version
danmar Jun 16, 2026
e4b2808
cleanup
danmar Jun 16, 2026
1c6d7dd
remove redundant semicolon
danmar Jun 16, 2026
82d06df
clang-tidy
danmar Jun 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .github/workflows/CI-unixish-docker.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ jobs:

strategy:
matrix:
image: ["ubuntu:24.04", "ubuntu:25.10", "alpine:3.23"]
image: ["ubuntu:24.04", "ubuntu:25.10", "alpine:3.23", "oraclelinux:8"]
fail-fast: false # Prefer quick result

runs-on: ubuntu-22.04
Expand All @@ -111,6 +111,12 @@ jobs:
run: |
apk add make g++ pcre-dev bash python3 libxml2-utils

- name: Install missing software on Oracle Linux
if: contains(matrix.image, 'oraclelinux')
run: |
yum install -y git python3 which epel-release
yum install -y cmake3 gcc-c++ make pcre-devel

# needs to be called after the package installation since
# - it doesn't call "apt-get update"
- name: ccache
Expand Down
2 changes: 1 addition & 1 deletion lib/checks.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,6 @@ namespace CheckInstances
{
/** List of registered check classes. This is used by Cppcheck to run checks and generate documentation */
CPPCHECKLIB const std::list<Check *>& get();
};
}

#endif // checksH
4 changes: 2 additions & 2 deletions lib/settings.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ Settings::~Settings() = default;
Settings::Settings(const Settings&) = default;
Settings & Settings::operator=(const Settings &) = default;

Settings::Settings(Settings&&) noexcept = default;
Settings & Settings::operator=(Settings &&) noexcept = default;
Settings::Settings(Settings&&) NOEXCEPT = default;
Settings & Settings::operator=(Settings &&) NOEXCEPT = default;

std::string Settings::loadCppcheckCfg(Settings& settings, Suppressions& suppressions, bool debug)
{
Expand Down
16 changes: 14 additions & 2 deletions lib/settings.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,18 @@
};


#if defined(__GNUC__) && !defined(__clang__) && __GNUC__ <= 9
// Hack to workaround GCC bug.
// Details: https://trac.cppcheck.net/ticket/14850
// seen on:
// oraclelinux:8, g++-8.5
// ubuntu:20.04, g++-9.4.0
#define NOEXCEPT
#else
#define NOEXCEPT noexcept
#endif


/**
* @brief This is just a container for general settings so that we don't need
* to pass individual values to functions or constructors now or in the
Expand All @@ -109,12 +121,12 @@
public:
Settings();
~Settings();

Check warning

Code scanning / Cppcheck Premium

User-provided copy and move member functions of a class should have appropriate signatures. Assignment operator must have lvalue ref-qualifier must not be virtual and have only one argument which must be non-volatile Warning

User-provided copy and move member functions of a class should have appropriate signatures. Assignment operator must have lvalue ref-qualifier must not be virtual and have only one argument which must be non-volatile
Settings(const Settings&);
Settings& operator=(const Settings&);

Settings(Settings&&) noexcept;
Settings& operator=(Settings&&) noexcept;
Settings(Settings&&) NOEXCEPT;

Check warning

Code scanning / Cppcheck Premium

Exception-unfriendly functions shall be noexcept Warning

Exception-unfriendly functions shall be noexcept

Check warning

Code scanning / Cppcheck Premium

User-provided copy and move member functions of a class should have appropriate signatures. Move constructor must be noexcept and have only one argument which must be non-volatile Warning

User-provided copy and move member functions of a class should have appropriate signatures. Move constructor must be noexcept and have only one argument which must be non-volatile
Settings& operator=(Settings&&) NOEXCEPT;

Check warning

Code scanning / Cppcheck Premium

Exception-unfriendly functions shall be noexcept Warning

Exception-unfriendly functions shall be noexcept

static std::string loadCppcheckCfg(Settings& settings, Suppressions& suppressions, bool debug = false);

Expand Down
Loading