Fix machine manifest tests

1 files changed, 7 insertions, 2 deletions

diff --git a/bbot/manifest.cxx b/bbot/manifest.cxx
index 52841f3..35e472c 100644
--- a/bbot/manifest.cxx
+++ b/bbot/manifest.cxx
@@ -625,8 +625,7 @@ namespace bbot
       throw parsing (p.name (), nv.name_line, nv.name_column, d);
     };
 
-    auto bad_value =
-      [&p, &nv] (const string& d, size_t offset = 0) [[noreturn]]
+    auto bad_value = [&p, &nv] (const string& d, size_t offset = 0)
     {
       throw parsing (p.name (), nv.value_line, nv.value_column + offset, d);
     };
@@ -642,6 +641,12 @@ namespace bbot
       {
         bad_value ("invalid " + what);
       }
+
+      // Can't be here. Would be redundant if it were possible to declare
+      // lambda with the [[noreturn]] attribute. Note that GCC (non-portably)
+      // supports that.
+      //
+      return result_status::abnormal;
     };
 
     // Make sure this is the start and we support the version.