Skip to content

WarpX.H: reorder inclusions#6795

Merged
RemiLehe merged 2 commits intoBLAST-WarpX:developmentfrom
lucafedeli88:warpx_header_reorder_inclusions
Apr 28, 2026
Merged

WarpX.H: reorder inclusions#6795
RemiLehe merged 2 commits intoBLAST-WarpX:developmentfrom
lucafedeli88:warpx_header_reorder_inclusions

Conversation

@lucafedeli88
Copy link
Copy Markdown
Member

This PR reorders the inclusions in the WarpX.H header, following the convention :

  • WarpX forward declaration headers
  • WarpX headers
  • ablastr headers
  • AMReX forward declaration headers
  • AMReX headers
  • STL headers

each group is in alphabetical order, except for:

#ifdef WARPX_USE_FFT
#   ifdef WARPX_DIM_RZ
#       include "BoundaryConditions/PML_RZ_fwd.H"
#       include "FieldSolver/SpectralSolver/SpectralSolverRZ_fwd.H"
#   else
#       include "FieldSolver/SpectralSolver/SpectralSolver_fwd.H"
#   endif
#endif

which is placed at the end of the WarpX forward declaration headers section.

@lucafedeli88 lucafedeli88 added the cleaning Clean code, improve readability label Apr 21, 2026
@RemiLehe RemiLehe merged commit 9b08157 into BLAST-WarpX:development Apr 28, 2026
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleaning Clean code, improve readability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants