diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2016-06-26 16:36:53 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2016-06-26 16:36:53 +0200 |
commit | 61617d7bb990f04970c6357d8e9a9095174d8fc4 (patch) | |
tree | c73f266c027d09dd1d0cd4d4233e8fc70ce27447 /build2/cli | |
parent | 4d1c02b736f4c1e827b11085cdc83ce4b46c03d1 (diff) |
Minor module interface cleanups
Diffstat (limited to 'build2/cli')
-rw-r--r-- | build2/cli/module.cxx | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/build2/cli/module.cxx b/build2/cli/module.cxx index 44b36a1..478c8a8 100644 --- a/build2/cli/module.cxx +++ b/build2/cli/module.cxx @@ -36,18 +36,13 @@ namespace build2 tracer trace ("cli::init"); l5 ([&]{trace << "for " << base.out_path ();}); - // Make sure the cxx module has been loaded since we need its - // targets types (?xx{}). Note that we don't try to load it - // ourselves because of the non-trivial variable merging - // semantics. So it is better to let the user load cxx - // explicitly. + // Make sure the cxx module has been loaded since we need its targets + // types (?xx{}). Note that we don't try to load it ourselves because of + // the non-trivial variable merging semantics. So it is better to let + // the user load cxx explicitly. // - { - auto l (base["cxx.loaded"]); - - if (!l || !cast<bool> (l)) - fail (loc) << "cxx module must be loaded before cli"; - } + if (!cast_false<bool> (base["cxx.loaded"])) + fail (loc) << "cxx module must be loaded before cli"; // Enter module variables. // |