src: remove empty name check in node_env_var.cc

RaisinTen · targos · commit a356f32e66df · 2021-05-01T09:44:42.000+02:00
The empty name check has been removed since this has landed: libuv/libuv#2473 PR-URL: #36133 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/src/node_env_var.cc b/src/node_env_var.cc
@@ -178,9 +178,7 @@ Local<Array> RealEnvStore::Enumerate(Isolate* isolate) const {
   for (int i = 0; i < count; i++) {
 #ifdef _WIN32
     // If the key starts with '=' it is a hidden environment variable.
-    // The '\0' check is a workaround for the bug behind
-    // https://github.com/libuv/libuv/pull/2473 and can be removed later.
-    if (items[i].name[0] == '=' || items[i].name[0] == '\0') continue;
+    if (items[i].name[0] == '=') continue;
 #endif
     MaybeLocal<String> str = String::NewFromUtf8(isolate, items[i].name);
     if (str.IsEmpty()) {
