[ET Trac] #2907: Arith uses very generic names for its (global) include files
Roland Haas
trac-noreply at einsteintoolkit.org
Fri Dec 12 13:52:37 CST 2025
#2907: Arith uses very generic names for its (global) include files
Reporter: Roland Haas
Status: new
Milestone:
Version:
Type: bug
Priority: minor
Component: CarpetX
Thorn `Arith` in `CarpetX` provides a number of helper functions for arithmetic operations. However it uses a large number of include files in its `interface.ccl` `INCLUDES HEADER` section. Since these file names are globally visible \(not even restriction to other thorns `REQUIRE`ing `Arith` or having a `USES HEADER` in their `interface.ccl`, this pollutes \(a\) global name space \(namely the `configs/sim/bindings/include/` which _all_ thorns use to search for include files using a `-I` compiler option\). Right now the files are:
```
INCLUDES HEADER: arr.hxx IN arr.hxx
INCLUDES HEADER: cons.hxx IN cons.hxx
INCLUDES HEADER: cplx.hxx IN cplx.hxx
INCLUDES HEADER: defs.hxx IN defs.hxx
INCLUDES HEADER: div.hxx IN div.hxx
INCLUDES HEADER: dual.hxx IN dual.hxx
INCLUDES HEADER: mat.hxx IN mat.hxx
INCLUDES HEADER: rational.hxx IN rational.hxx
INCLUDES HEADER: rten.hxx IN rten.hxx
INCLUDES HEADER: simd.hxx IN simd.hxx
INCLUDES HEADER: smallvector.hxx IN smallvector.hxx
INCLUDES HEADER: spvect.hxx IN spvect.hxx
INCLUDES HEADER: sum.hxx IN sum.hxx
INCLUDES HEADER: ten3.hxx IN ten3.hxx
INCLUDES HEADER: tuple.hxx IN tuple.hxx
INCLUDES HEADER: vec.hxx IN vec.hxx
INCLUDES HEADER: vect.hxx IN vect.hxx
```
of which `vec.hxx` and `vect.hxx` seem to be the most problematic.
It may be a good idea to consolidate these into fewer files and / or prefixing with `arith_` or so to create a namespace.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2907/arith-uses-very-generic-names-for-its
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.einsteintoolkit.org/pipermail/trac/attachments/20251212/15cb0a7d/attachment.htm>
More information about the Trac
mailing list