_normalize_path: removed redundant table.remove in discarding drive path logic which I did not notice.

Author: shikorido

lua/plenary/path.lua

@@ -138,15 +138,23 @@ local function _normalize_path(filename, cwd)
 
   local has = string.find(filename, path.sep .. "..", 1, true) or string.find(filename, ".." .. path.sep, 1, true)
 
+  --qqq
+  --vim.notify("_normalize_path: has = " .. vim.inspect(has), vim.log.levels.WARN)
+  --!qqq
+
   if has then
     local is_abs = is_absolute(filename, path.sep)
+    --qqq
+    --vim.notify("_normalize_path: is_abs = " .. vim.inspect(is_abs), vim.log.levels.WARN)
+    --!qqq
     local split_without_disk_name = function(filename_local)
       local parts = _split_by_separator(filename_local)
       -- Remove disk name part on Windows
       if is_abs and (path.sep == "\\" or U.is_msys2) then
+        --qqq
+        --vim.notify("_normalize_path: parts = " .. vim.inspect(parts), vim.log.levels.WARN)
+        --!qqq
         table.remove(parts, 1)
-        table.remove(parts, 1)
-        --table.remove(parts, 1)
       end
       return parts
     end
@@ -188,6 +196,10 @@ local function _normalize_path(filename, cwd)
   return out_file
 end
 
+--qqq
+--vim.notify("_normalize_path: call returned " .. vim.inspect(_normalize_path("C:\\msys64\\home\\..\\123", "C:\\msys64\\123")), vim.log.levels.WARN)
+--!qqq
+
 local clean = function(pathname)
   if is_uri(pathname) then
     return pathname

lua/plenary/utils.lua

@@ -92,26 +92,27 @@ function M.posix_to_windows(posix_path)
   -- Idk if WezTerm paths behaviour leaks to msys2 actually...
   if not prefix_changed then
     if #posix_path == 2 and posix_path:find("/[A-Za-z]") then
-      --vim.notify("Only '[A-Za-z]:' case: " .. posix_path, vim.log.levels.WARN)
+      --vim.notify("Only '/[A-Za-z]' case: " .. posix_path, vim.log.levels.WARN)
       posix_path = posix_path:gsub("^/([A-Za-z])", "%1:\\")
       prefix_changed = true
     else
-      --vim.notify("'[A-Za-z]:?/' case: " .. posix_path, vim.log.levels.WARN)
+      --vim.notify("'/[A-Za-z]:?/' case: " .. posix_path, vim.log.levels.WARN)
       posix_path = posix_path:gsub("^/([A-Za-z]):?/", "%1:\\")
       prefix_changed = true
     end
   end
 
-  -- Lets try to be msys2 compliant for testing.
+  -- Lets try to use posix-style paths for testing.
   --posix_path = posix_path:gsub("^/([A-Za-z]):?([^0-9A-Za-z_-]?)", "/%1%2")
 
 
-  -- Replace remaining forward slashes with backslashes.
+  -- Replace remaining backslashes with forward slashes.
   posix_path = posix_path:gsub("/", "\\")
 
   -- For bash.exe (nvim shell) it is better to use backslashes
-  -- cause forward slashes must be escaped. Need to come up with something
-  -- cause nvim for windows (even clang64 binary) prefers windows-style paths (with backslashes?).
+  -- cause forward slashes must to be escaped. Need to come up with something
+  -- cause nvim for windows (even clang64 binary) prefers windows-style paths (with backslashes as well?).
+  -- UPD. "set shellslash" makes the trick by converting forward slashes to backslashes in shell invocations.
   --posix_path = posix_path:gsub("\\", "/")