ci, style: clang-format and macOS

Fix clang-format changes.
Go back down to the free tier for macOS.
Update code-ql action.

.github/workflows/codeql-analysis.yml

@@ -48,7 +48,7 @@ jobs:
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v2
+      uses: github/codeql-action/init@v3
       with:
         languages: ${{ matrix.language }}
         # If you wish to specify custom queries, you can do so here or in a config file.

.github/workflows/macos.yml

@@ -14,7 +14,7 @@ concurrency:
 
 jobs:
   build:
-    runs-on: macos-latest-xlarge
+    runs-on: macos-latest
 
     steps:
     - uses: actions/checkout@v3

include/Ergodic_moves_3.hpp

@@ -333,9 +333,8 @@ namespace ergodic_moves
   /// @param manifold The simplicial manifold
   /// @param candidate The vertex to check
   /// @returns True if (6,2) move is possible
-  [[nodiscard]] inline auto is_62_movable(Manifold const&      manifold,
-                                          Vertex_handle const& candidate)
-      -> bool
+  [[nodiscard]] inline auto is_62_movable(
+      Manifold const& manifold, Vertex_handle const& candidate) -> bool
   {
     if (manifold.dimensionality() != 3)
     {
@@ -485,8 +484,8 @@ namespace ergodic_moves
   /// @see
   /// https://github.com/CGAL/cgal/blob/8430d04539179f25fb8e716f99e19d28589beeda/TDS_3/include/CGAL/Triangulation_data_structure_3.h#L2094
   [[nodiscard]] inline auto incident_cells_from_edge(
-      Delaunay_t<3> const& triangulation, Edge_handle const& edge)
-      -> std::optional<Cell_container>
+      Delaunay_t<3> const& triangulation,
+      Edge_handle const&   edge) -> std::optional<Cell_container>
   {
     if (!triangulation.tds().is_edge(edge.first, edge.second, edge.third))
     {
@@ -519,8 +518,8 @@ namespace ergodic_moves
   /// @param t_edge_candidate The edge to check
   /// @returns A container of incident cells if there are exactly 4 or nullopt
   [[nodiscard]] inline auto find_bistellar_flip_location(
-      Delaunay const& triangulation, Edge_handle const& t_edge_candidate)
-      -> std::optional<Cell_container>
+      Delaunay const&    triangulation,
+      Edge_handle const& t_edge_candidate) -> std::optional<Cell_container>
   {
     if (auto incident_cells =
             incident_cells_from_edge(triangulation, t_edge_candidate);
@@ -565,10 +564,9 @@ namespace ergodic_moves
   /// @param top Top vertex of the cells being flipped
   /// @param bottom Bottom vertex of the cells being flipped
   /// @returns A flipped triangulation or nullopt
-  [[nodiscard]] inline auto bistellar_flip(Delaunay    triangulation,
-                                           Edge_handle edge, Vertex_handle top,
-                                           Vertex_handle bottom)
-      -> std::optional<Delaunay>
+  [[nodiscard]] inline auto bistellar_flip(
+      Delaunay triangulation, Edge_handle edge, Vertex_handle top,
+      Vertex_handle bottom) -> std::optional<Delaunay>
   {
     // Get the cells incident to the edge
     auto incident_cells = get_incident_cells(triangulation, edge);
@@ -841,10 +839,9 @@ namespace ergodic_moves
   /// @param t_after The manifold after the move
   /// @param t_move The type of move
   /// @return True if the move correctly changed the triangulation
-  [[nodiscard]] inline auto check_move(Manifold const&                t_before,
-                                       Manifold const&                t_after,
-                                       move_tracker::move_type const& t_move)
-      -> bool
+  [[nodiscard]] inline auto check_move(
+      Manifold const& t_before, Manifold const& t_after,
+      move_tracker::move_type const& t_move) -> bool
   {
     switch (t_move)
     {

include/Foliated_triangulation.hpp

@@ -901,7 +901,7 @@ namespace foliated_triangulations
       {
         causal_vertices.emplace_back(*gen++, i + 1);
       }  // j
-    }    // i
+    }  // i
     return causal_vertices;
   }  // make_foliated_ball
 

include/Geometry.hpp

@@ -99,7 +99,7 @@ struct [[nodiscard("This contains data!")]] Geometry<3>
     swap(swap_from.N1_SL, swap_into.N1_SL);
     swap(swap_from.N0, swap_into.N0);
   }  // swap
-};   // struct Geometry<3>
+};  // struct Geometry<3>
 
 using Geometry_3 = Geometry<3>;
 

include/Metropolis.hpp

@@ -493,7 +493,7 @@ class MoveStrategy<Strategies::METROPOLIS, ManifoldType>
           m_failed_moves.four_four_moves());
     }
   }  // print_results
-};   // Metropolis
+};  // Metropolis
 
 using Metropolis_3 =
     MoveStrategy<Strategies::METROPOLIS, manifolds::Manifold_3>;

include/Move_tracker.hpp

@@ -79,7 +79,7 @@ namespace move_tracker
     if (dim == 3) { return NUMBER_OF_3D_MOVES; }
     if (dim == 4) { return NUMBER_OF_4D_MOVES; }
     return 0;  // Error condition
-  }            // moves_per_dimension
+  }  // moves_per_dimension
 
   /**
    * \brief The data and methods to track ergodic moves

include/Utilities.hpp

@@ -49,8 +49,8 @@ enum class topology_type
 /// @param t_os The output stream
 /// @param t_topology The topology
 /// @returns An output string of the topology
-inline auto operator<<(std::ostream& t_os, topology_type const& t_topology)
-    -> std::ostream&
+inline auto operator<<(std::ostream&        t_os,
+                       topology_type const& t_topology) -> std::ostream&
 {
   switch (t_topology)
   {
@@ -90,13 +90,11 @@ namespace utilities
   /// @param t_initial_radius The radius of the first foliation t=1
   /// @param t_foliation_spacing The spacing between foliations
   /// @returns A filename
-  [[nodiscard]] inline auto make_filename(topology_type const& t_topology,
-                                          Int_precision        t_dimension,
-                                          Int_precision t_number_of_simplices,
-                                          Int_precision t_number_of_timeslices,
-                                          double        t_initial_radius,
-                                          double t_foliation_spacing) noexcept
-      -> std::filesystem::path
+  [[nodiscard]] inline auto make_filename(
+      topology_type const& t_topology, Int_precision t_dimension,
+      Int_precision t_number_of_simplices, Int_precision t_number_of_timeslices,
+      double t_initial_radius,
+      double t_foliation_spacing) noexcept -> std::filesystem::path
   {
     std::string filename;
     if (t_topology == topology_type::SPHERICAL) { filename += "S"; }