improve how deletes are read by avoiding temporary object creation (#… #671
hendrikmuhsAnnotations
46 warnings
|
Debug on macos-latest
The following actions uses node12 which is deprecated and will be forced to run on node16: lukka/run-cmake@v3. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Debug on macos-latest
The following actions uses Node.js version which is deprecated and will be forced to run on node20: hendrikmuhs/[email protected], lukka/run-cmake@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Debug on macos-latest:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L57
variable 'files_added' set but not used [-Wunused-but-set-variable]
|
|
Debug on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Debug on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Debug on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Debug on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Debug on macos-latest:
keyvi/tests/keyvi/util/active_object_test.cpp#L49
unused variable 'last_call' [-Wunused-variable]
|
|
Debug on macos-latest:
keyvi/tests/keyvi/util/active_object_test.cpp#L68
unused variable 'last_call' [-Wunused-variable]
|
|
Debug on macos-latest
The process '/opt/homebrew/bin/git' failed with exit code 128
|
|
Release on macos-latest
The following actions uses node12 which is deprecated and will be forced to run on node16: lukka/run-cmake@v3. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Release on macos-latest
The following actions uses Node.js version which is deprecated and will be forced to run on node20: hendrikmuhs/[email protected], lukka/run-cmake@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Release on macos-latest:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L57
variable 'files_added' set but not used [-Wunused-but-set-variable]
|
|
Release on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Release on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Release on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Release on macos-latest:
keyvi/include/keyvi/index/internal/tiered_merge_policy.h#L68
variable 'totalSizeToMerge' set but not used [-Wunused-but-set-variable]
|
|
Release on macos-latest:
keyvi/tests/keyvi/util/active_object_test.cpp#L49
unused variable 'last_call' [-Wunused-variable]
|
|
Release on macos-latest:
keyvi/tests/keyvi/util/active_object_test.cpp#L68
unused variable 'last_call' [-Wunused-variable]
|
|
Release on macos-latest
The process '/opt/homebrew/bin/git' failed with exit code 128
|
|
Debug on ubuntu-22.04
The following actions uses node12 which is deprecated and will be forced to run on node16: lukka/run-cmake@v3. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Debug on ubuntu-22.04
The following actions uses Node.js version which is deprecated and will be forced to run on node20: hendrikmuhs/[email protected], lukka/run-cmake@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L43
all parameters should be named in a function [hicpp-named-parameter,readability-named-parameter]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L44
100 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L48
function 'compile_multiple' has cognitive complexity of 26 (threshold 25) [readability-function-cognitive-complexity]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L95
2 adjacent parameters of 'finalize_compile' of similar type ('const std::string &') are easily swapped by mistake [bugprone-easily-swappable-parameters]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L110
the parameter 'line' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L113
statement should be inside braces [hicpp-braces-around-statements,readability-braces-around-statements]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L117
variable 'value' is not initialized [cppcoreguidelines-init-variables]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L136
the parameter 'line' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L139
statement should be inside braces [hicpp-braces-around-statements,readability-braces-around-statements]
|
|
Debug on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L143
variable 'value' is not initialized [cppcoreguidelines-init-variables]
|
|
Debug on ubuntu-22.04
The process '/usr/bin/git' failed with exit code 128
|
|
Release on ubuntu-22.04
The following actions uses node12 which is deprecated and will be forced to run on node16: lukka/run-cmake@v3. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Release on ubuntu-22.04
The following actions uses Node.js version which is deprecated and will be forced to run on node20: hendrikmuhs/[email protected], lukka/run-cmake@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Release on ubuntu-22.04
The process '/usr/bin/git' failed with exit code 128
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L43
all parameters should be named in a function [hicpp-named-parameter,readability-named-parameter]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L44
100 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L48
function 'compile_multiple' has cognitive complexity of 26 (threshold 25) [readability-function-cognitive-complexity]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L95
2 adjacent parameters of 'finalize_compile' of similar type ('const std::string &') are easily swapped by mistake [bugprone-easily-swappable-parameters]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L110
the parameter 'line' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L113
statement should be inside braces [hicpp-braces-around-statements,readability-braces-around-statements]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L117
variable 'value' is not initialized [cppcoreguidelines-init-variables]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L136
the parameter 'line' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L139
statement should be inside braces [hicpp-braces-around-statements,readability-braces-around-statements]
|
|
Release on ubuntu-22.04:
keyvi/bin/keyvicompiler/keyvicompiler.cpp#L143
variable 'value' is not initialized [cppcoreguidelines-init-variables]
|