From 0cf707721ea3b9cb0a6137fb4ba1109b726644dc Mon Sep 17 00:00:00 2001 From: Wolfgang Bangerth Date: Fri, 14 Nov 2003 14:08:52 +0000 Subject: [PATCH] Add a TODO. git-svn-id: https://svn.dealii.org/trunk@8183 0785d39b-7218-0410-832d-ea1e28bc413d --- deal.II/deal.II/source/fe/fe.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/deal.II/deal.II/source/fe/fe.cc b/deal.II/deal.II/source/fe/fe.cc index 117c8d32d7..eabd70b3bd 100644 --- a/deal.II/deal.II/source/fe/fe.cc +++ b/deal.II/deal.II/source/fe/fe.cc @@ -468,6 +468,7 @@ compute_2nd (const Mapping &mapping, InternalDataBase &fe_internal, FEValuesData &data) const { +//TODO[GK]: This function presently has a flaw: it ignores the offset parameter. This means that if it is called for faces or subfaces, it computes the data for _all_ faces or subfaces, even though we only need to have this for one of them. given that computing second derivatives is expensive, this is significant. What should be done is to honor the offset parameter and of course only compute what we actually need Assert ((fe_internal.update_each | fe_internal.update_once) & update_second_derivatives, ExcInternalError()); -- 2.39.5