Feature updated core

• provide container methods such as front(), back(), push_back() etc that are commonly used in the STL containers. This should help with swapping code between OpenFOAM and other projects and (in the future) make the code easier to read for newcomers.
• reduce the use of SLList in a few more places to avoid repeated memory allocations within loops
• provide point/vector dist() and distSqr() methods for differences. This avoids intermediate variables and looping (should be more cache friendly).
• new mesh access method cellBb to return the boundBox of a mesh cell. Uses cached cellPoints (if available) or calculates from the face points. Eliminates similar code fragments that were scattered about the code
• improved functionality and efficiency for boundBox and treeBoundBox
• more consistent naming and handling of indexedOctree and treeData components.

Changes were originally rolled into MR !568 (merged), but grew too large for that.

changed milestone to %v2212

added enhancement label

requested review from @kuti

assigned to @andy

mentioned in merge request !568 (merged)

added 22 commits

• b7cec1d1 - ENH: linked-lists accept more familiar STL method names
• 06f31d99 - ENH: use CircularBuffer instead SLList for FIFO-style handling
• 5426f52f - ENH: add front(), back() methods to List containers
• 5f16af7c - ENH: add FixedList templated get<unsigned>() methods
• bfe722c9 - ENH: face::connected() method
• bf6e602b - STYLE: use stack-like naming for ifEntry handling
• 5688dbea - ENH: base classes for wave algorithms (manage non-templated parts)
• 75ebcd45 - ENH: vector mag(), magSqr() methods - complementary to dist(), distSqr()
• e14ce41e - ENH: add primitiveMesh cellBb()
• 6eb38637 - ENH: general boundBox/treeBoundBox improvements
• 5463e227 - ENH: use simpler boundBox handling
• 50b27c31 - ENH: split off template-invariant part of indexedOctree node indexing
• 293fe503 - STYLE: octree pushPoint with local constexpr (more readable code)
• 05ec79b3 - ENH: simplify sub-octant bound-box search
• ce591166 - ENH: reduce some internal overhead when splitting octree nodes (#2609 (closed))
• 0af398bf - ENH: setter/getter for octree permutation tolerance
• 77b0b00b - ENH: octree findBox, findSphere with external storage of results
• d95f5b22 - ENH: cleanup treeData items (#2609 (closed))
• f5055ab1 - ENH: add boundBox/triangle intersection test to boundBox
• f99a6497 - ENH: replace triangleFuncs::intersectBb usage with boundBox::intersect
• 1181dc87 - BUG: avoid infinite recursion in AABBTree (fixes #2616 (closed))
• edfd23b7 - ENH: use simpler constructor forms for treeData types

Compare with previous version

changed the description

marked this merge request as ready

added 9 commits

• 3bdbf6ad - ENH: simplify sub-octant bound-box search
• afe9e25e - ENH: reduce some internal overhead when splitting octree nodes (#2609 (closed))
• ea51970a - ENH: setter/getter for octree permutation tolerance
• 72c0008c - ENH: octree findBox, findSphere with external storage of results
• cc644fa1 - ENH: cleanup treeData items (#2609 (closed))
• 4ed53f5c - ENH: add boundBox/triangle intersection test to boundBox
• 86e620bb - ENH: replace triangleFuncs::intersectBb usage with boundBox::intersect
• cc5cf1ad - BUG: avoid infinite recursion in AABBTree (fixes #2616 (closed))
• 1c2c3a0e - ENH: use simpler constructor forms for treeData types

Compare with previous version

• linux64ClangDPInt32Opt (clang13)
• linux64GccDPInt32Opt
• linux64GccSPDPInt64Debug
• Alltest: No new error

approved this merge request

approved this merge request

added 28 commits

• 1c2c3a0e...9f9b8fb6 - 6 commits from branch develop
• f3ba6c6d - ENH: linked-lists accept more familiar STL method names
• db882651 - ENH: use CircularBuffer instead SLList for FIFO-style handling
• c7e6ae30 - ENH: add front(), back() methods to List containers
• d3e285b4 - ENH: add FixedList templated get<unsigned>() methods
• e4139898 - ENH: face::connected() method
• c33167dc - STYLE: use stack-like naming for ifEntry handling
• 5ec435ac - ENH: base classes for wave algorithms (manage non-templated parts)
• 27c2cdc0 - ENH: vector mag(), magSqr() methods - complementary to dist(), distSqr()
• 0ba458fd - ENH: add primitiveMesh cellBb()
• 1339c335 - ENH: general boundBox/treeBoundBox improvements
• e5006a62 - ENH: use simpler boundBox handling
• 3384747f - ENH: split off template-invariant part of indexedOctree node indexing
• fc9311ba - STYLE: octree pushPoint with local constexpr (more readable code)
• b1294462 - ENH: simplify sub-octant bound-box search
• 3d7dc6a8 - ENH: reduce some internal overhead when splitting octree nodes (#2609 (closed))
• b8d01a88 - ENH: setter/getter for octree permutation tolerance
• f638db48 - ENH: octree findBox, findSphere with external storage of results
• ac4f580d - ENH: cleanup treeData items (#2609 (closed))
• 38b663b6 - ENH: add boundBox/triangle intersection test to boundBox
• ffbad655 - ENH: replace triangleFuncs::intersectBb usage with boundBox::intersect
• fc0b9803 - BUG: avoid infinite recursion in AABBTree (fixes #2616 (closed))
• 98598ba0 - ENH: use simpler constructor forms for treeData types

Compare with previous version

mentioned in commit 2984d1e3

merged

mentioned in issue #2609 (closed)