Moving some TODO comments to MR discussion.

lanl · Sep 30, 2024 · 8ab5205 · 8ab5205
1 parent cac4be1
commit 8ab5205
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/spiner/databox.hpp b/spiner/databox.hpp
@@ -156,9 +156,6 @@ class DataBox {
   PORTABLE_INLINE_FUNCTION T get_data_value(Args... args) const {
       return Transform::reverse(dataView_(std::forward<Args>(args)...));
   }
-  // TODO: Should this method be const?  Having this be const is in line with the second operator()
-  //       below (the one with const), which allows the user to modify the dependent variable
-  //       values of a const DataBox.
   template<typename... Args>
   PORTABLE_INLINE_FUNCTION void set_data_value(const T new_value, Args... args) const {
     dataView_(std::forward<Args>(args)...) = Transform::forward(new_value);