From: wolf Date: Mon, 12 Jul 2004 13:54:43 +0000 (+0000) Subject: Add 2 TODOs for Guido. X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2fef5ba0fb54c9fa49feec084c1a6b2ea3e18d33;p=dealii-svn.git 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 f6217c9717..e626c0131f 100644 --- 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 &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 &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.)