From b2fc1fb4a13ffa58640333a3909dd0e53bd21995 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Wed, 31 May 2017 16:29:33 +0200
Subject: Adjust extension semantics in file rule to match
 search_existing_file()

---
 build2/target.cxx | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

(limited to 'build2/target.cxx')

diff --git a/build2/target.cxx b/build2/target.cxx
index 1b6757c..3ee9850 100644
--- a/build2/target.cxx
+++ b/build2/target.cxx
@@ -530,8 +530,8 @@ namespace build2
 
   // path_target
   //
-  const string& path_target::
-  derive_extension (const char* de, bool search)
+  const string* path_target::
+  derive_extension (bool search, const char* de)
   {
     // See also search_existing_file() if updating anything here.
     //
@@ -541,7 +541,7 @@ namespace build2
       // Note that returning by reference is now MT-safe since once the
       // extension is specified, it is immutable.
       //
-      return *p;
+      return p;
     else
     {
       optional<string> e;
@@ -559,10 +559,15 @@ namespace build2
         if (de != nullptr)
           e = de;
         else
+        {
+          if (search)
+            return nullptr;
+
           fail << "no default extension for target " << *this;
+        }
       }
 
-      return ext (move (*e));
+      return &ext (move (*e));
     }
   }
 
-- 
cgit v1.1