FESystem has a number of functions that are called in the member initializer list
of the constructors. These are all static functions, of course, and don't access
any member variables.
This patch moves them ouf of the class altogether. This has the advantage that
it reduces the size of the .h file, and that it reduces the number of functions
exported by the shared libraries. Rather, these functions may as well live
inside an anonymous namespace in the .cc file.
There are no functional changes, nor anything that would be visible to the user.
The value of this argument was unused by this class, but several derived
classes passed something down anyway. Fix this by simply removing the argument.
Timo Heister [Sat, 23 Jan 2016 10:56:27 +0000 (11:56 +0100)]
fix test codim_one/error_estimator_02
fixes dealii/tests/codim_one/error_estimator_02.cc:75:9: warning: using
integer absolute value function 'abs' when argument is of floating point
type [-Wabsolute-value]
if (abs(p(1)-1.0)<1e-5)
Timo Heister [Thu, 21 Jan 2016 20:47:08 +0000 (15:47 -0500)]
doxygen: do not export duplicate functions
Since we started handing .cc files to doxygen, every explicit template
declaration of a function inside a .cc would generate another line for
that function in the class documentation (not clickable and before the
correct ones). We fix this by hiding undocumented functions. A quick
test shows, that even with this change, undocumented functions inside
classes defined in the .h will still be exported, because things like
@name groups and @ingroup count as "documentation".
David Wells [Wed, 20 Jan 2016 18:50:58 +0000 (13:50 -0500)]
Reindent comments in the documentation: part 2.
Up until 801cfbfc1d deal.II used a style where comments would start at
about the 20th character for visibility. Some comments within the
documentation were not updated at the time.
Timo Heister [Tue, 19 Jan 2016 03:27:23 +0000 (22:27 -0500)]
setup missing parallel level neighbors
Level ghosts were not set up if neighbor is finer and we have no cells
on that finest level. This causes a situation where
Triangulation::level_ghost_owners is not symmetric causing a deadlock
(and maybe also wrong information down the line). This causes a few test
output changes...
David Wells [Tue, 19 Jan 2016 02:10:39 +0000 (21:10 -0500)]
Improve ParameterHandler continuation handling.
This commit builds on 23f306dfc5. ParameterHandler now correctly handles
continuation lines in both versions of the read_input function, while before it
only handled continuation lines in one version (the file name as a string
version) before.
David Wells [Mon, 18 Jan 2016 20:43:23 +0000 (15:43 -0500)]
Improve some ParameterHandler code samples.
The code samples are now formatted in the same way as the rest of the
library. Additionally, I fixed a memory leak in one example and got rid
of 'void main' in another.
David Wells [Mon, 18 Jan 2016 14:09:21 +0000 (09:09 -0500)]
Switch from the old comment format to the new one.
Up until 801cfbfc1d deal.II used a style where comments would start at
about the 20th character for visibility. Some comments within the
documentation were not updated at the time.
David Wells [Sat, 16 Jan 2016 18:00:50 +0000 (13:00 -0500)]
Assign unique numbers to GMSH elements.
GMSH will not properly load information from a .msh file if some
element (geometry element, as in line, face, quadrilateral, hexahedron)
indices are repeated.
Timo Heister [Sun, 17 Jan 2016 19:46:27 +0000 (13:46 -0600)]
table layout and image fixes
- fix weird case where doxygen shifts the last cell in a table downward
causing them to be not correctly aligned
- cleanup some image layout
- do not break image tags (this will break various scripts)
Denis Davydov [Thu, 14 Jan 2016 19:18:59 +0000 (20:18 +0100)]
various fixes and improvements to SLEPc unit tests
some fiddling with solver settings (preconditioner, tolerance, etc) to
produce the same output on OS-X and Linux.
/slepc/step-36_parallel still needs two different outputs, though.
fixed an uninitialised initial vector bug in slepc/step-36 unit test