Skip to content

WIP: Canal/clang format

Philippe Canal requested to merge canal/clang-format into master

I made a few tweak to the .clang-format and uploaded for comparison only a massive apply of it to the whole VecGeom (this is only a 190,000 lines patch :) ) so that we can spot any additional issues.

The major issue I see is that clang-format does not seem be configurable to break before a return type and thus we end up with the sub-optimal:

  template <bool ForInnerSurface>
  VECGEOM_CUDA_HEADER_BOTH VECGEOM_INLINE bool PointOnHyperbolicSurface(Vector3D<Precision> const &p) const

rather than

  template <bool ForInnerSurface>
  VECGEOM_CUDA_HEADER_BOTH
  VECGEOM_INLINE
  bool PointOnHyperbolicSurface(Vector3D<Precision> const &p) const

This is actually not really in the coding covention per se but is much more readable.

Merge request reports