Apply suggestions from code review

Co-authored-by: wraymo <[email protected]>
y-scope · Aug 2, 2024 · 0f6b354 · 0f6b354
1 parent 0176ecb
commit 0f6b354
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/components/core/src/clp_s/ArchiveWriter.cpp b/components/core/src/clp_s/ArchiveWriter.cpp
@@ -161,7 +161,6 @@ size_t ArchiveWriter::store_tables() {
      * of the metadata in the "schema_metadata" vector as we compress the tables. The metadata is
      * flushed once all of the schema tables have been compressed.
      */
-
     using schema_map_it = decltype(m_id_to_schema_writer)::iterator;
     std::vector<schema_map_it> schemas;
     std::vector<std::tuple<size_t, size_t>> table_metadata;

diff --git a/components/core/src/clp_s/TableReader.hpp b/components/core/src/clp_s/TableReader.hpp
@@ -66,7 +66,7 @@ class TableReader {
      */
     void read_table(size_t table_id, std::shared_ptr<char[]>& buf, size_t& buf_size);
 
-    size_t get_uncompressed_table_size(size_t table_id) const {
+    [[nodiscard]] size_t get_uncompressed_table_size(size_t table_id) const {
         return m_table_metadata.at(table_id).uncompressed_size;
     }