From dc2bb76f03ecf9a57c2e26531ed4a439201399ee Mon Sep 17 00:00:00 2001
From: Timo Koch <timokoch@uio.no>
Date: Wed, 29 Jan 2025 03:13:14 +0100
Subject: [PATCH] [gridwriter] Fix return type in forwarding function
This makes the function work for std::vector<bool>. T is bool
but values[0] returns (non-standard-compliant in some impl.) some proxy reference class
even if the container is const. Explicitly converting to T (or generally for ranges
std::ranges::range_value_t) works.
Thed default return type would be auto which also returns by value.
---
dumux/io/gridwriter.hh | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/dumux/io/gridwriter.hh b/dumux/io/gridwriter.hh
index 60914aea27..7e2666afa4 100644
--- a/dumux/io/gridwriter.hh
+++ b/dumux/io/gridwriter.hh
@@ -363,7 +363,7 @@ class OutputModule : private GridWriter<typename GridVariables::GridGeometry::Gr
if (values.size() != gridVariables_.gridGeometry().vertexMapper().size())
DUNE_THROW(Dune::InvalidStateException, "Given container does not match the number of points in the grid");
- this->setPointField(name, [&] (const auto& vertex) {
+ this->setPointField(name, [&] (const auto& vertex) -> std::ranges::range_value_t<C> {
return values[gridVariables_.gridGeometry().vertexMapper().index(vertex)];
}, prec);
}
@@ -391,7 +391,7 @@ class OutputModule : private GridWriter<typename GridVariables::GridGeometry::Gr
if (values.size() != gridVariables_.gridGeometry().elementMapper().size())
DUNE_THROW(Dune::InvalidStateException, "Given container does not match the number of cells in the grid");
- this->setCellField(name, [&] (const auto& element) {
+ this->setCellField(name, [&] (const auto& element) -> std::ranges::range_value_t<C> {
return values[gridVariables_.gridGeometry().elementMapper().index(element)];
}, prec);
}
--
GitLab