Skip to content
Limestone-CI

Refactor code to address clang-tidy warnings #951

Sign in to view logs

Refactor code to address clang-tidy warnings

Refactor code to address clang-tidy warnings #951

Triggered via push February 18, 2025 07:39
@umeganeumegane
pushed 41550b6
wip/blob
Status Failure
Total duration 1m 20s
Artifacts –

ci-build.yml

on: push
Analysis
1m 10s
Analysis
Matrix: Build
Fit to window
Zoom out
Zoom in

Annotations

1 error and 28 warnings
Analysis
modernize-pass-by-value: src/limestone/compaction_options.h#L34
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L36
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L48
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L49
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L61
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L62
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L64
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L77
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L78
pass by value and use std::move
modernize-pass-by-value: src/limestone/compaction_options.h#L80
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L34
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L36
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L48
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L49
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L61
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L62
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L64
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L77
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L78
pass by value and use std::move
modernize-pass-by-value: src/./limestone/compaction_options.h#L80
pass by value and use std::move
modernize-pass-by-value: src/./limestone/dblog_scan.h#L86
pass by value and use std::move
modernize-pass-by-value: src/./limestone/dblog_scan.h#L98
pass by value and use std::move
performance-unnecessary-value-param: src/./limestone/dblog_scan.h#L98
the parameter 'options' is copied for each invocation but only used as a const reference; consider making it a const reference
modernize-pass-by-value: src/limestone/dblog_scan.h#L86
pass by value and use std::move
modernize-pass-by-value: src/limestone/dblog_scan.h#L98
pass by value and use std::move
performance-unnecessary-value-param: src/limestone/dblog_scan.h#L98
the parameter 'options' is copied for each invocation but only used as a const reference; consider making it a const reference
performance-unnecessary-value-param: src/limestone/datastore_snapshot.cpp#L303
the parameter 'options' is copied for each invocation but only used as a const reference; consider making it a const reference
performance-unnecessary-copy-initialization: src/limestone/datastore_snapshot.cpp#L307
the variable 'to_dir' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference