From: Wolfgang Bangerth Date: Mon, 12 Jul 2004 13:54:43 +0000 (+0000) Subject: Add 2 TODOs for Guido. X-Git-Tag: v8.0.0~14963 X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f3fc858c0e7e867222f048bbda9f95c2a99c422f;p=dealii.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.)