LLM Code Review Checklist - C++ part Mid-2026
- Dominant language
- C++
- Stars
- 1.2k
- Forks
- 261
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 71
Description
Nested checklist of all `*.cpp` / `*.h` file pairs under `cpp/dolfinx/`. Also add links to PRs when done.
- [ ] **common/**
- [ ] `defines.h`
- [ ] `dolfinx_common.h`
- [ ] `dolfinx_doc.h`
- [ ] `IndexMap`
- [ ] `IndexMap.h`
- [ ] `IndexMap.cpp`
- [ ] `local_range.h`
- [ ] `log`
- [ ] `log.h`
- [ ] `log.cpp`
- [ ] `math.h`
- [ ] `MPI`
- [ ] `MPI.h`
- [ ] `MPI.cpp`
- [ ] `Scatterer.h`
- [ ] `sort.h`
- [ ] `Table`
- [ ] `Table.h`
- [ ] `Table.cpp`
- [ ] `TimeLogger`
- [ ] `TimeLogger.h`
- [ ] `TimeLogger.cpp`
- [ ] `Timer.h`
- [ ] `timing`
- [ ] `timing.h`
- [ ] `timing.cpp`
- [ ] `types.h`
- [ ] `utils.h` https://github.com/FEniCS/dolfinx/pull/4322
- [ ] `version.h`
- [ ] **fem/**
- [ ] `assemble_expression_impl.h`
- [ ] `assemble_matrix_impl.h`
- [ ] `assemble_scalar_impl.h`
- [ ] `assemble_vector_impl.h`
- [ ] `assembler.h`
- [ ] `Constant.h`
- [ ] `CoordinateElement`
- [ ] `CoordinateElement.h`
- [ ] `CoordinateElement.cpp`
- [ ] `DirichletBC`
- [ ] `DirichletBC.h`
- [ ] `DirichletBC.cpp`
- [ ] `discreteoperators.h`
- [ ] `DofMap`
- [ ] `DofMap.h`
- [ ] `DofMap.cpp`
- [ ] `dofmapbuilder`
- [ ] `dofmapbuilder.h`
- [ ] `dofmapbuilder.cpp`
- [ ] `dolfinx_fem.h`
- [ ] `ElementDofLayout`
- [ ] `ElementDofLayout.h`
- [ ] `ElementDofLayout.cpp`
- [ ] `Expression.h`
- [ ] `FiniteElement`
- [ ] `FiniteElement.h`
- [ ] `FiniteElement.cpp`
- [ ] `Form.h`
- [ ] `Function.h`
- [ ] `FunctionSpace.h`
- [ ] `interpolate.h`
- [ ] `kernel.h`
- [ ] `pack.h`
- [ ] `petsc`
- [ ] `petsc.h`
- [ ] `petsc.cpp`
- [ ] `sparsitybuild`
- [ ] `sparsitybuild.h`
- [ ] `sparsitybuild.cpp`
- [ ] `traits.h`
- [ ] `utils`
- [ ] `utils.h`
- [ ] `utils.cpp`
- [ ] **geometry/**
- [ ] `BoundingBoxTree.h`
- [ ] `dolfinx_geometry.h`
- [ ] `gjk.h`
- [ ] `utils.h`
- [ ] **graph/**
- [ ] `AdjacencyList.h`
- [ ] `dolfinx_graph.h`
- [ ] `ordering`
- [ ] `ordering.h`
- [ ] `ordering.cpp`
- [ ] `partition`
- [ ] `partition.h`
- [ ] `partition.cpp`
- [ ] `partitioners`
- [ ] `partitioners.h`
- [ ] `partitioners.cpp`
- [ ] `utils`
- [ ] `utils.h`
- [ ] `utils.cpp`
- [ ] **io/**
- [ ] `ADIOS2Writers`
- [ ] `ADIOS2Writers.h`
- [ ] `ADIOS2Writers.cpp`
- [ ] `cells`
- [ ] `cells.h`
- [ ] `cells.cpp`
- [ ] `dolfinx_io.h`
- [ ] `HDF5Interface`
- [ ] `HDF5Interface.h`
- [ ] `HDF5Interface.cpp`
- [ ] `utils.h`
- [ ] `vtk_utils`
- [ ] `vtk_utils.h`
- [ ] `vtk_utils.cpp`
- [ ] `VTKFile`
- [ ] `VTKFile.h`
- [ ] `VTKFile.cpp`
- [ ] `VTKHDF.h`
- [ ] `xdmf_function`
- [ ] `xdmf_function.h`
- [ ] `xdmf_function.cpp`
- [ ] `xdmf_mesh`
- [ ] `xdmf_mesh.h`
- [ ] `xdmf_mesh.cpp`
- [ ] `xdmf_utils`
- [ ] `xdmf_utils.h`
- [ ] `xdmf_utils.cpp`
- [ ] `XDMFFile`
- [ ] `XDMFFile.h`
- [ ] `XDMFFile.cpp`
- [ ] **la/**
- [ ] `dolfinx_la.h`
- [ ] `matmul.h`
- [ ] `matrix_csr_impl.h`
- [ ] `MatrixCSR.h`
- [ ] `mattrans.h`
- [ ] `petsc`
- [ ] `petsc.h`
- [ ] `petsc.cpp`
- [ ] `slepc`
- [ ] `slepc.h`
- [ ] `slepc.cpp`
- [ ] `SparsityPattern`
- [ ] `SparsityPattern.h`
- [ ] `SparsityPattern.cpp`
- [ ] `superlu_dist`
- [ ] `superlu_dist.h`
- [ ] `superlu_dist.cpp`
- [ ] `utils.h`
- [ ] `Vector.h`
- [ ] **mesh/**
- [ ] `cell_types`
- [ ] `cell_types.h`
- [ ] `cell_types.cpp`
- [ ] `dolfinx_mesh.h`
- [ ] `EntityMap`
- [ ] `EntityMap.h`
- [ ] `EntityMap.cpp`
- [ ] `generation.h`
- [ ] `Geometry.h`
- [ ] `graphbuild`
- [ ] `graphbuild.h`
- [ ] `graphbuild.cpp`
- [ ] `Mesh.h`
- [ ] `MeshTags.h`
- [ ] `permutationcomputation`
- [ ] `permutationcomputation.h`
- [ ] `permutationcomputation.cpp`
- [ ] `Topology`
- [ ] `Topology.h`
- [ ] `Topology.cpp`
- [ ] `topologycomputation`
- [ ] `topologycomputation.h`
- [ ] `topologycomputation.cpp`
- [ ] `utils`
- [ ] `utils.h`
- [ ] `utils.cpp`
- [ ] **nls/**
- [ ] `dolfinx_nls.h`
- [ ] `NewtonSolver`
- [ ] `NewtonSolver.h`
- [ ] `NewtonSolver.cpp`
- [ ] **refinement/**
- [ ] `dolfinx_refinement.h`
- [ ] `interval.h`
- [ ] `mark.h`
- [ ] `option.h`
- [ ] `plaza`
- [ ] `plaza.h`
- [ ] `plaza.cpp`
- [ ] `refine.h`
- [ ] `uniform`
- [ ] `uniform.h`
- [ ] `uniform.cpp`
- [ ] `utils`
- [ ] `utils.h`
- [ ] `utils.cpp`
- [ ] **(top level)**
- [ ] `dolfinx.h`
Contributor guide
Research direction
Review the listed headers and source files under cpp/dolfinx/, beginning with the unchecked entries in common/ and the existing utils.h PR link. Determine the review criteria from the project context before proceeding. Done means the applicable checklist entries are completed and links to resulting PRs are added.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- developer-experience
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100