]> https://gitweb.dealii.org/ - dealii-svn.git/commitdiff
Add a TODO.
authorwolf <wolf@0785d39b-7218-0410-832d-ea1e28bc413d>
Fri, 14 Nov 2003 14:08:52 +0000 (14:08 +0000)
committerwolf <wolf@0785d39b-7218-0410-832d-ea1e28bc413d>
Fri, 14 Nov 2003 14:08:52 +0000 (14:08 +0000)
git-svn-id: https://svn.dealii.org/trunk@8183 0785d39b-7218-0410-832d-ea1e28bc413d

deal.II/deal.II/source/fe/fe.cc

index 117c8d32d73afab36935bd2649e2660a6d331935..eabd70b3bd78b62a5bfa44b3e64f6f79dfbff697 100644 (file)
@@ -468,6 +468,7 @@ compute_2nd (const Mapping<dim>                   &mapping,
             InternalDataBase                     &fe_internal,
             FEValuesData<dim>                    &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());

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.