aboutsummaryrefslogtreecommitdiff
path: root/build2/cli
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2016-06-26 16:36:53 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2016-06-26 16:36:53 +0200
commit61617d7bb990f04970c6357d8e9a9095174d8fc4 (patch)
treec73f266c027d09dd1d0cd4d4233e8fc70ce27447 /build2/cli
parent4d1c02b736f4c1e827b11085cdc83ce4b46c03d1 (diff)
Minor module interface cleanups
Diffstat (limited to 'build2/cli')
-rw-r--r--build2/cli/module.cxx17
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.
//