From 038496f8345d7b83dee0b144c84c61314cf23b11 Mon Sep 17 00:00:00 2001
From: Pascal Engeler <engelerp@phys.ethz.ch>
Date: Wed, 15 Jan 2020 15:03:40 +0100
Subject: [PATCH] Const correctness
---
include/coupler.hpp | 2 +-
include/coupler_simple.hpp | 4 ++--
include/driver.hpp | 2 +-
include/driver_simple.hpp | 4 ++--
4 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/coupler.hpp b/include/coupler.hpp
index e2b50b8..ed35420 100644
--- a/include/coupler.hpp
+++ b/include/coupler.hpp
@@ -11,7 +11,7 @@ class Coupler{
virtual void precompute(const value_t t_end, const value_t dt, const std::vector<drum_t>& drum_vec) noexcept = 0;
virtual void step(value_t dt) noexcept = 0;
- virtual value_t operator()(size_t drum_index, size_t neighbour_index) const noexcept = 0;
+ virtual value_t operator()(const size_t drum_index, const size_t neighbour_index) const noexcept = 0;
};
#endif
diff --git a/include/coupler_simple.hpp b/include/coupler_simple.hpp
index 8bf7997..03b7542 100644
--- a/include/coupler_simple.hpp
+++ b/include/coupler_simple.hpp
@@ -7,7 +7,7 @@
template <typename value_t, typename drum_t>
class CouplerSimple: public Coupler<value_t, drum_t>{
public:
- CouplerSimple(value_t amplitude, value_t frequency): amplitude_(amplitude), frequency_(frequency), time_(0.) {}
+ CouplerSimple(const value_t amplitude, const value_t frequency): amplitude_(amplitude), frequency_(frequency), time_(0.) {}
~CouplerSimple() = default;
void precompute(const value_t t_end, const value_t dt, const std::vector<drum_t>& drum_vec) noexcept final override { }
@@ -17,7 +17,7 @@ class CouplerSimple: public Coupler<value_t, drum_t>{
time_ += dt;
}
- value_t operator()(size_t drum_index, size_t neighbour_index) const noexcept final override
+ value_t operator()(const size_t drum_index, const size_t neighbour_index) const noexcept final override
{
return amplitude_*(std::sin(frequency_*time_) + 1.);
}
diff --git a/include/driver.hpp b/include/driver.hpp
index 2630c8c..b1d4217 100644
--- a/include/driver.hpp
+++ b/include/driver.hpp
@@ -12,7 +12,7 @@ class Driver{
virtual void precompute(const value_t t_end, const value_t dt, const std::vector<drum_t>& drum_vec) noexcept = 0;
virtual void step(value_t dt) noexcept = 0;
- virtual value_t operator()(size_t drum_index) const noexcept = 0;
+ virtual value_t operator()(const size_t drum_index) const noexcept = 0;
};
#endif
diff --git a/include/driver_simple.hpp b/include/driver_simple.hpp
index 6b57505..29d3c90 100644
--- a/include/driver_simple.hpp
+++ b/include/driver_simple.hpp
@@ -7,7 +7,7 @@
template<typename value_t, typename drum_t>
class DriverSimple: public Driver<value_t, drum_t>{
public:
- DriverSimple(value_t amplitude, value_t frequency): amplitude_(amplitude), frequency_(frequency), time_(0.) {}
+ DriverSimple(const value_t amplitude, const value_t frequency): amplitude_(amplitude), frequency_(frequency), time_(0.) {}
~DriverSimple() = default;
void precompute(const value_t t_end, const value_t dt, const std::vector<drum_t>& drum_vec) noexcept final override{ }
@@ -16,7 +16,7 @@ class DriverSimple: public Driver<value_t, drum_t>{
time_ += dt;
}
- value_t operator()(size_t drum_index) const noexcept final override{
+ value_t operator()(const size_t drum_index) const noexcept final override{
return amplitude_*(std::sin(frequency_*2.*M_PI*time_) + 1.);
}
--
GitLab