From b39ceb27794272100154b88b6d50195d1dca0431 Mon Sep 17 00:00:00 2001 From: "Kartik K. Agaram" Date: Wed, 22 Apr 2015 13:29:02 -0700 Subject: 1133 - more reorg of commandline parsing 1. Drop the ability to run just some C++ tests. It's a lousy interface to use line numbers, we can't selectively run mu tests, we haven't used it in a while because our tests run plenty fast, and it's complicating other parts, like Next_recipe_number and test space handling. 2. Create a new layer right up top to show the usage message and all the different forms that are possible. Good for documentation until we come up with a way to put the different forms in their own layers. At least it's out of the test layer now. 3. Strengthen the assertion that no recipes exist in test space before we start running any sorts of tests. Earlier it was possible for files loaded explicitly to overflow into test space because we were asserting before load, not after. Now we check if we need to run tests, load all files, then make the assertion, run tests, and exit if necessary. Now we don't need to mess with commandline args at all in layer 50. That's a sign that we're on the right track. --- cpp/020run | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'cpp/020run') diff --git a/cpp/020run b/cpp/020run index 0ce86d7a..3dad12bf 100644 --- a/cpp/020run +++ b/cpp/020run @@ -93,13 +93,16 @@ inline bool done(routine& rr) { return running_at(rr) >= steps(rr).size(); } -:(before "End Main") +:(before "End Commandline Parsing") if (argc > 1) { - setup(); for (int i = 1; i < argc; ++i) { load(argv[i]); } +} +:(before "End Main") +if (!Run_tests) { + setup(); Trace_stream = new trace_stream; //? Trace_stream->dump_layer = "all"; //? 2 transform_all(); -- cgit 1.4.1-2-gfad0