From 8f2070a23dee739c83ac9af55c1fe98a49fcd31a Mon Sep 17 00:00:00 2001 From: Andrej Babic Date: Mon, 12 Feb 2018 11:46:27 +0100 Subject: [PATCH] Improve const correctness --- src/H5Writer.cpp | 2 +- src/H5Writer.hpp | 2 +- src/WriterManager.cpp | 10 +++++----- src/WriterManager.hpp | 8 ++++---- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/H5Writer.cpp b/src/H5Writer.cpp index 6147b31..6dd41ba 100644 --- a/src/H5Writer.cpp +++ b/src/H5Writer.cpp @@ -195,7 +195,7 @@ void H5Writer::create_file(hsize_t frame_chunk) current_frame_chunk = frame_chunk; } -bool H5Writer::is_file_open() +bool H5Writer::is_file_open() const { return (file.getId() != -1); } diff --git a/src/H5Writer.hpp b/src/H5Writer.hpp index dd5d3a6..6f5412d 100644 --- a/src/H5Writer.hpp +++ b/src/H5Writer.hpp @@ -33,7 +33,7 @@ class H5Writer public: H5Writer(const std::string& filename, hsize_t frames_per_file=0, hsize_t initial_dataset_size=1000, hsize_t dataset_increase_step=1000); virtual ~H5Writer(); - bool is_file_open(); + bool is_file_open() const; void close_file(); void write_data(const std::string& dataset_name, const size_t data_index, const char* data, const std::vector& data_shape, const size_t data_bytes_size, const std::string& data_type, const std::string& endianness); diff --git a/src/WriterManager.cpp b/src/WriterManager.cpp index ea0e02c..4c4e053 100644 --- a/src/WriterManager.cpp +++ b/src/WriterManager.cpp @@ -49,12 +49,12 @@ string WriterManager::get_status() } } -string WriterManager::get_output_file() +string WriterManager::get_output_file() const { return output_file; } -map WriterManager::get_statistics() +map WriterManager::get_statistics() const { map result = {{"n_received_frames", n_received_frames.load()}, {"n_written_frames", n_written_frames.load()}, @@ -96,7 +96,7 @@ void WriterManager::set_parameters(const map& new_parameters #endif } -const map& WriterManager::get_parameters_type() +const map& WriterManager::get_parameters_type() const { return parameters_type; } @@ -111,7 +111,7 @@ bool WriterManager::is_running() return running_flag.load(); } -bool WriterManager::is_killed() +bool WriterManager::is_killed() const { return killed_flag.load(); } @@ -131,7 +131,7 @@ void WriterManager::lost_frame(size_t frame_index) n_lost_frames++; } -bool WriterManager::are_all_parameters_set() +bool WriterManager::are_all_parameters_set() { lock_guard lock(parameters_mutex); diff --git a/src/WriterManager.hpp b/src/WriterManager.hpp index f2b3c6a..ab680b6 100644 --- a/src/WriterManager.hpp +++ b/src/WriterManager.hpp @@ -32,16 +32,16 @@ class WriterManager void stop(); void kill(); bool is_running(); - bool is_killed(); + bool is_killed() const; std::string get_status(); bool are_all_parameters_set(); - std::string get_output_file(); + std::string get_output_file() const; - const std::map& get_parameters_type(); + const std::map& get_parameters_type() const; std::map get_parameters(); void set_parameters(const std::map& new_parameters); - std::map get_statistics(); + std::map get_statistics() const; void received_frame(size_t frame_index); void written_frame(size_t frame_index); void lost_frame(size_t frame_index);