clang-tidy: filter out subproject headers

Subproject headers should be filtered out so that when building
outside of the CI docker container (or Yocto) we do not get tons
of clang-tidy failures for

clang-tidy: filter out subproject headers

Subproject headers should be filtered out so that when building
outside of the CI docker container (or Yocto) we do not get tons
of clang-tidy failures for things present in those subprojects.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I35e909d667ce700e6c68ff3c2f269f1ef549dca3

show more ...

clang-tidy: disable readability-suspicious-call-ar

This commit disables the readability-suspicious-call-argument check
in Clang-Tidy configuration to avoid false positives errors

'''
../dump-exte

clang-tidy: disable readability-suspicious-call-ar

This commit disables the readability-suspicious-call-argument check
in Clang-Tidy configuration to avoid false positives errors

'''
../dump-extensions/openpower-dumps/dump_manager_system.cpp:94:18: error: 2nd argument 'size' (passed to 'dumpSize')
looks like it might be swapped with the 3rd, 'dumpId' (passed to 'sourceId')
[readability-suspicious-call-argument,-warnings-as-errors]
94 | upEntry->update(timeStamp, size, dumpId);
| ^ ~~~~ ~~~~~~
../dump-extensions/openpower-dumps/system_dump_entry.hpp:78:10: note: in the call to 'update', declared here
78 | void update(uint64_t timeStamp, uint64_t dumpSize, const uint32_t sourceId)
| ^ ~~~~~~~~ ~~~~~~~~
'''

Change-Id: I39ff4d4002373cee8c1a17f2c416c563d259a74a
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: disable cppcoreguidelines-avoid-non-c

Disabled the `cppcoreguidelines-avoid-non-const-global-variables`
check due to dependency on libpldm library API that requires
non-const global vari

clang-tidy: disable cppcoreguidelines-avoid-non-c

Disabled the `cppcoreguidelines-avoid-non-const-global-variables`
check due to dependency on libpldm library API that requires
non-const global variables.

Change-Id: I65af4735dfc88f0dce8ca20477f9466eb8b5e174
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable modernize-use-equals-default

Modified code to address issues flagged by this check.

Tested: Build and unit tests passed successfully.

Change-Id: I17f31ea24c59e910ff4249b62334ed7

clang-tidy: Enable modernize-use-equals-default

Modified code to address issues flagged by this check.

Tested: Build and unit tests passed successfully.

Change-Id: I17f31ea24c59e910ff4249b62334ed7f4e38c20c
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Add modernize checks

These changes improve code readability, maintainability, and
performance by leveraging modern C++ features and best practices.

Tested: Build and unit tests passed s

clang-tidy: Add modernize checks

These changes improve code readability, maintainability, and
performance by leveraging modern C++ features and best practices.

Tested: Build and unit tests passed successfully.

Change-Id: Ia26044323c8e67feb039868b9bff93069983481b
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable readability-implicit-bool

Modified code to address issues flagged by
readability-implicit-bool-conversion check.

Tested: Build and unit tests passed successfully.

Change-Id: Ic7

clang-tidy: Enable readability-implicit-bool

Modified code to address issues flagged by
readability-implicit-bool-conversion check.

Tested: Build and unit tests passed successfully.

Change-Id: Ic7a15466ea628629eb4a98233b12523fbbbe794d
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable readability checks on repo

The readability-* checks in Clang-Tidy focus on improving code
readability by enforcing guidelines and suggesting improvements
that can enhance the clar

clang-tidy: Enable readability checks on repo

The readability-* checks in Clang-Tidy focus on improving code
readability by enforcing guidelines and suggesting improvements
that can enhance the clarity and maintainability of C++ code.

Tested: Build and unit tests passed successfully.

Change-Id: Icf996f860bc763e0aebd095d8f50c67813e2395e
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Add performance checks

The checks are valuable for improving the runtime efficiency and
resource utilization of your programs.

Tested: Build and unit tests passed successfully.

Change-

clang-tidy: Add performance checks

The checks are valuable for improving the runtime efficiency and
resource utilization of your programs.

Tested: Build and unit tests passed successfully.

Change-Id: Ia4150edfb5c38794d1d4a9b75bdb67d1ed5aa3fe
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: clang-tidy: Add few misc checks

Checks that encompass various miscellaneous guidelines and rules
that don't fall into more specific categories like readability,
performance, or correctne

clang-tidy: clang-tidy: Add few misc checks

Checks that encompass various miscellaneous guidelines and rules
that don't fall into more specific categories like readability,
performance, or correctness. These checks often focus on code
quality, stylistic preferences, or potential pitfalls that are
not covered by other specific checks.

Tested: Build and unit tests passed successfully.

Change-Id: I6dd76a66cd3dd76a76b285398ce74311241b5827
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable cppcoreguidelines-pro-bounds

Modified code to address issues flagged by
cppcoreguidelines-pro-bounds-array-to-pointer-decay check.

Tested: Build and unit tests passed successfull

clang-tidy: Enable cppcoreguidelines-pro-bounds

Modified code to address issues flagged by
cppcoreguidelines-pro-bounds-array-to-pointer-decay check.

Tested: Build and unit tests passed successfully.

Change-Id: I68f418d0dad440df685c103709589f6e78d141fb
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Add few cppcoreguidelines check

These checks cover a wide range of best practices and
recommendations aimed at improving the safety, readability,
maintainability, and performance of our

clang-tidy: Add few cppcoreguidelines check

These checks cover a wide range of best practices and
recommendations aimed at improving the safety, readability,
maintainability, and performance of our C++ codebase according
to the C++ Core Guidelines.

Tested: Build and unit tests passed successfully.

Change-Id: I4f2986556626e0792d3e5b4aa3704ed9895ff32b
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Add clang-analyzer checks

This checks primary purpose is to identify potential bugs,security
vulnerabilities, and adherence to coding standards by examining
the source code without execu

clang-tidy: Add clang-analyzer checks

This checks primary purpose is to identify potential bugs,security
vulnerabilities, and adherence to coding standards by examining
the source code without executing it.

Tested: Build and unit tests passed successfully.

Change-Id: Ie4a9e6319484c149e13a6f63227e2462fe0be425
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Add cert checks

This update enhances code security by incorporating CERT guideline
checks using clang-tidy. These checks are designed to identify
potential security vulnerabilities and e

clang-tidy: Add cert checks

This update enhances code security by incorporating CERT guideline
checks using clang-tidy. These checks are designed to identify
potential security vulnerabilities and ensure compliance with
secure coding standards.

Tested: Build and unit tests passed successfully.

Change-Id: Ic292a71f89a23d17c8dfdb3b3398ae82d3bc78cb
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable bugprone-unchecked-optional

Modified code to address issues flagged by
bugprone-unchecked-optional-access check.

Tested: Build and unit tests passed successfully.

Change-Id: Icd

clang-tidy: Enable bugprone-unchecked-optional

Modified code to address issues flagged by
bugprone-unchecked-optional-access check.

Tested: Build and unit tests passed successfully.

Change-Id: Icd445afaf827f7c346e44dfe3ce9ba98374e176a
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable bugprone-empty-catch

Modified code to address issues flagged by this check.

Tested: Build and unit tests passed successfully.

Change-Id: I19ea0ffe4861f5a5e0aea118b4c20f6faf1d2c3

clang-tidy: Enable bugprone-empty-catch

Modified code to address issues flagged by this check.

Tested: Build and unit tests passed successfully.

Change-Id: I19ea0ffe4861f5a5e0aea118b4c20f6faf1d2c35
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable bugprone-narrowing-conversions

Modified code to address issues flagged by this check.

Tested: Build and unit tests passed successfully.

Change-Id: Idd5e3701725a833324535d29f0bff

clang-tidy: Enable bugprone-narrowing-conversions

Modified code to address issues flagged by this check.

Tested: Build and unit tests passed successfully.

Change-Id: Idd5e3701725a833324535d29f0bfffb3ba180e88
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Enable bugprone-forward-declaration

Modified code to address issues flagged by
bugprone-forward-declaration-namespace check.

Tested: Build and unit tests passed successfully.

Change-Id

clang-tidy: Enable bugprone-forward-declaration

Modified code to address issues flagged by
bugprone-forward-declaration-namespace check.

Tested: Build and unit tests passed successfully.

Change-Id: Ied7e54a6aa30d3f8110bcd7036c84b99e7fe6fb8
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Add bugprone checks

Enable bugprone checks to identify problematic patterns that could
lead to bugs or undefined behavior.

Tested: Build and unit tests passed successfully.

Change-Id:

clang-tidy: Add bugprone checks

Enable bugprone checks to identify problematic patterns that could
lead to bugs or undefined behavior.

Tested: Build and unit tests passed successfully.

Change-Id: If15d84e1735980b893147b1135a3e20038a43d46
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: Initial Commit

This commit introduces clang-tidy and modifies code to address
issues flagged by the checks. Additionally, it includes the
initial infrastructure setup.

Tested: Build and

clang-tidy: Initial Commit

This commit introduces clang-tidy and modifies code to address
issues flagged by the checks. Additionally, it includes the
initial infrastructure setup.

Tested: Build and unit tests completed successfully.

Change-Id: I55f9f4a2a9a1b1cd2016773b47935484d6a57867
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...