From 29de23d8ccdff22dd6fcbf079afa9f3150fd9f09 Mon Sep 17 00:00:00 2001
From: Oliver Sander <sander@igpm.rwth-aachen.de>
Date: Fri, 14 Oct 2011 16:14:36 +0000
Subject: [PATCH] add a remark for later

[[Imported from SVN: r7922]]
---
 dune/gfe/localgfetestfunction.hh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/dune/gfe/localgfetestfunction.hh b/dune/gfe/localgfetestfunction.hh
index da3ecd78..40e55a23 100644
--- a/dune/gfe/localgfetestfunction.hh
+++ b/dune/gfe/localgfetestfunction.hh
@@ -74,6 +74,8 @@ void LocalGFETestFunction<dim,ctype,LocalFiniteElement,TargetSpace>::evaluateFun
         
         Dune::FieldMatrix< double, embeddedDim, embeddedDim > derivative;
         
+        /** \todo This call internally keeps computing the value of the gfe function at 'local'.
+         * This is expensive.  Eventually we should precompute it once and reused the result. */
         localGFEFunction_.evaluateDerivativeOfValueWRTCoefficient (local,
                                                                    i,
                                                                    derivative);
@@ -95,6 +97,8 @@ void LocalGFETestFunction<dim,ctype,LocalFiniteElement,TargetSpace>::evaluateJac
     
     for (size_t i=0; i<size(); i++) {
         
+        /** \todo This call internally keeps computing the value of the gfe function at 'local'.
+         * This is expensive.  Eventually we should precompute it once and reused the result. */
         Tensor3< double, embeddedDim, embeddedDim, dim > derivative;
         localGFEFunction_.evaluateDerivativeOfGradientWRTCoefficient (local,
                                                                    i,
-- 
GitLab