Add test for handling IO errors in compaction_catalog class #662
Annotations
2 errors and 9 warnings
Build (ubuntu-24.04)
Process completed with exit code 1.
|
Analysis
- *<https://github.com/project-tsurugi/limestone/runs/29869988941?check_suite_focus=true|[Clang-Tidy] 9 warnings>*
|
modernize-use-nodiscard:
include/limestone/api/limestone_exception.h#L34
function 'error_code' should be marked [[nodiscard]]
|
cppcoreguidelines-macro-usage:
include/limestone/api/limestone_exception.h#L58
function-like macro 'THROW_LIMESTONE_EXCEPTION' used; consider a 'constexpr' template function
|
cppcoreguidelines-macro-usage:
include/limestone/api/limestone_exception.h#L61
function-like macro 'THROW_LIMESTONE_IO_EXCEPTION' used; consider a 'constexpr' template function
|
cppcoreguidelines-special-member-functions,hicpp-special-member-functions:
src/limestone/file_operations.h#L41
class 'file_operations' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
|
readability-inconsistent-declaration-parameter-name:
src/limestone/file_operations.h#L104
function 'limestone::internal::real_file_operations::open_ifstream' has a definition with different parameter names
|
cppcoreguidelines-avoid-non-const-global-variables:
src/limestone/compaction_catalog.h#L209
variable 'file_ops_' is non-const and globally accessible, consider making it const
|
fuchsia-statically-constructed-objects:
src/limestone/compaction_catalog.cpp#L298
static objects are disallowed; if possible, use a constexpr constructor instead
|
cert-err58-cpp:
src/limestone/compaction_catalog.cpp#L298
initialization of 'file_ops_' with static storage duration may throw an exception that cannot be caught
|
cppcoreguidelines-avoid-non-const-global-variables:
src/limestone/compaction_catalog.cpp#L298
variable 'file_ops_' is non-const and globally accessible, consider making it const
|