Add 2 TODOs for Guido.

git-svn-id: https://svn.dealii.org/trunk@9502 0785d39b-7218-0410-832d-ea1e28bc413d

diff --git a/deal.II/deal.II/source/dofs/dof_tools.cc b/deal.II/deal.II/source/dofs/dof_tools.cc
index f6217c9717260e2d61e23bcf419b02888338c39c..e626c0131f3208123b9f91ab9e3e1d916b006756 100644 (file)
--- a/deal.II/deal.II/source/dofs/dof_tools.cc
+++ b/deal.II/deal.II/source/dofs/dof_tools.cc
@@ -503,6 +503,8 @@ DoFTools::make_flux_sparsity_pattern (const DoFHandler<dim> &dof,
                                                 endc = dof.end();
 
 
+//TODO[GK]: would someone explain what the letters 'f', 'a', and 'n' mean?
+//Even better would, of course, be to change this to an enum
   Table<2,char> int_dof_mask(total_dofs, total_dofs);
   Table<2,char> flux_dof_mask(total_dofs, total_dofs);
 
@@ -535,7 +537,10 @@ DoFTools::make_flux_sparsity_pattern (const DoFHandler<dim> &dof,
                );
           Assert (jj < dof.get_fe().n_components(),
                   ExcInternalError());          
-         
+
+//TODO[GK]: the documentation only says that non-zeroness counts, but here
+//individual values are tested. this should be documented. Even better, rather
+//than passing in a matrix of doubles, we should pass in a Table<2,some-enum>
          if (int_mask(ii,jj) != 0)
            int_dof_mask(i,j) = 'f';
          if (flux_mask(ii,jj) == 1.)