From b882f365e9aebf95c07c8667e38ae2730931f74e Mon Sep 17 00:00:00 2001
From: Alexander Karatarakis <alkarata@microsoft.com>
Date: Tue, 14 Feb 2017 15:35:34 -0800
Subject: System::get_environmental_variable() now returns optional<>

---
 toolsrc/src/vcpkg_System.cpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'toolsrc/src/vcpkg_System.cpp')

diff --git a/toolsrc/src/vcpkg_System.cpp b/toolsrc/src/vcpkg_System.cpp
index e7349df69..3b056a675 100644
--- a/toolsrc/src/vcpkg_System.cpp
+++ b/toolsrc/src/vcpkg_System.cpp
@@ -73,17 +73,20 @@ namespace vcpkg::System
         std::cout << "\n";
     }
 
-    std::wstring get_environmental_variable(const wchar_t* varname) noexcept
+    optional<std::wstring> get_environmental_variable(const wchar_t* varname) noexcept
     {
         std::wstring ret;
         wchar_t* buffer;
         _wdupenv_s(&buffer, nullptr, varname);
-        if (buffer != nullptr)
+
+        if (buffer == nullptr)
         {
-            ret = buffer;
-            free(buffer);
+            return nullptr;
         }
-        return ret;
+
+        ret = buffer;
+        free(buffer);
+        return std::make_unique<std::wstring>(ret);
     }
 
     void set_environmental_variable(const wchar_t* varname, const wchar_t* varvalue) noexcept
-- 
cgit v1.2.3