currently bootstrap just uses strings to represent various compiletest arguments, such as mode.
obviously it would be better to use enums for these, as they are much less vulnerable to typos.
compiletest already has enum definitions for these (in common.rs), and the two already have a shared dependency (build_helper), and build_helper already has access to serde.
i would be willing to work on this refactor, if people think it would be a good idea.
currently bootstrap just uses strings to represent various compiletest arguments, such as mode.
obviously it would be better to use enums for these, as they are much less vulnerable to typos.
compiletestalready has enum definitions for these (incommon.rs), and the two already have a shared dependency (build_helper), andbuild_helperalready has access toserde.i would be willing to work on this refactor, if people think it would be a good idea.