cleanup for clang-tidy

project-tsurugi · Dec 12, 2024 · cdd79d5 · cdd79d5
1 parent 20a1999
commit cdd79d5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/include/limestone/api/datastore.h b/include/limestone/api/datastore.h
@@ -77,7 +77,7 @@ class datastore {
     /**
      * @brief destruct the object
      */
-    ~datastore() noexcept;
+    virtual ~datastore() noexcept;
 
     datastore(datastore const& other) = delete;
     datastore& operator=(datastore const& other) = delete;

diff --git a/src/limestone/rotation_result.cpp b/src/limestone/rotation_result.cpp
@@ -31,9 +31,8 @@ const std::set<boost::filesystem::path>& rotation_result::get_rotation_end_files
     return rotation_end_files;
 }
 
-void rotation_result::add_rotated_file(const std::string filename) {
+void rotation_result::add_rotated_file(const std::string& filename) {
     latest_rotated_files_.insert(filename);
-
 }
 
 } // namespace limestone::api
diff --git a/src/limestone/rotation_result.h b/src/limestone/rotation_result.h
@@ -32,13 +32,13 @@ namespace limestone::api {
 
 class rotation_result {
 public:
-    rotation_result(epoch_id_type epoch) : epoch_id_(epoch) {}
+    explicit rotation_result(epoch_id_type epoch) : epoch_id_(epoch) {}
 
     [[nodiscard]] epoch_id_type get_epoch_id() const;
     [[nodiscard]] const std::set<boost::filesystem::path>& get_rotation_end_files() const;
     void set_rotation_end_files(const std::set<boost::filesystem::path>& files);
+    void add_rotated_file(const std::string& filename);
 
-    void add_rotated_file(std::string);
 private:
     // A set of filenames that were rotated in this rotation process.
     std::set<std::string> latest_rotated_files_;