diff options
author | swhite <schnetter@cct.lsu.edu> | 2004-11-19 18:23:00 +0000 |
---|---|---|
committer | swhite <schnetter@cct.lsu.edu> | 2004-11-19 18:23:00 +0000 |
commit | 3f9c169f9357c1d537ec8b44f6904e6d279f7fca (patch) | |
tree | 9d40a3401d15e8f33e1296abf8daf5b3bcce56ca /Carpet/CarpetLib/src/dh.cc | |
parent | 68ef15f21f964d18aa8aa12f4b21ef32077ac2a9 (diff) |
unglob
I broke up dh<D>::recompose( bool ) into bite-sized chunks.
I made use of member function pointers to cut down as many as three
levels of blocks. See 'foreach_reflevel_component_mglevel'.
Resulting in:
1) the code of the main function 'recompose' now presents an outline of
its functionality, rather than a post-ice-age deluge.
2) component functions are named according to some comment found in the
code. For those who want to minimise comments.
3) most of the functions conform to my "rule of thumb" (comprehensible
chunks of code are no longer than my thumb)
4) nesting substantially reduced (The functions that still have more
than 5 levels of nested blocks are mostly those that are also
unthumbworthy. More room on the screen, but still room for
improvement.)
Probably some of these functions could have been non-members, but it
was more mechanical to consider them all as members.
darcs-hash:20041119182315-32473-7a328297ba05b013194c23ae5d5b516c1105b7c1.gz
Diffstat (limited to 'Carpet/CarpetLib/src/dh.cc')
0 files changed, 0 insertions, 0 deletions