Redesign new environment variables to use env_var_t instead of wcstring

Migrate uses of empty() to missing() to distinguish between empty variable and unset variable

1 files changed, 4 insertions, 5 deletions

diff --git a/function.cpp b/function.cpp
index e8452fa6..d0ac36b7 100644
--- a/function.cpp
+++ b/function.cpp
@@ -152,11 +152,10 @@ static void autoload_names( std::set<wcstring> &names, int get_hidden )
 {
 	size_t i;
 	
-	const wcstring path_var_wstr =  env_get_string( L"fish_function_path" );
-	const wchar_t *path_var = path_var_wstr.empty()?NULL:path_var_wstr.c_str(); 
-
-	if( ! path_var )
-		return;
+	const env_var_t path_var_wstr =  env_get_string( L"fish_function_path" );
+    if (path_var_wstr.missing())
+        return;
+	const wchar_t *path_var = path_var_wstr.c_str(); 
 	
     wcstring_list_t path_list;