Closed bretbrownjr closed 2 months ago
@dcbaker This touches all the interesting files, so it will of course be a pain to rebase. I'm doing this now because I'll be travelling a lot in the coming weeks and it would be nice to get this refactoring out of the way before then. That will make it easier for me to juggle batches of PRs that don't step on one another.
Oops. I agree. I thought that was in my ignore file. I'll amend and update the commit.
Including
#include
statements likecps/*.hpp
CPS_*
lowercase
instead ofUPPERCASE
to avoid collisions with preprocessor definitions from other projects.Incidental Changes
To make this commit less awkward, some incidental design changes were scoped in, including adding a
src/cps
subdirectory along withmeson.build
files to describe that subdirectory in a more encapsulated way for the rest of the build configuration.HAVE_UNREACHABLE
was renamed toCPS_USE_BUILTIN_UNREACHABLE
.main
was reimplemented to be a pass-through call to a newcps_config::run
function. This added a namespace to thecps-config
program itself that is distinct from the namespace for thecps
library.Incidental changes to include sorting, quoting, and spacing were added to make automated code formatters behave well with current include sorting and spelling patterns.
Updated some copyright notices.
Justification
This will better prepare the project to publish a
libcps
in the future, which will be desireable.Specifically:
utils.hpp
unreachable
.SIMPLE
.