From 0ea7421ae6618fe8efe9e3afc8fdcf7a4d89739a Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Fri, 11 Dec 2020 20:10:50 +0100 Subject: patch 8.2.2129: MS-Windows: Checking if a file name is absolute is slow Problem: MS-Windows: Checking if a file name is absolute is slow. Solution: Do not use mch_FullName(). (closes #7033) --- src/os_mswin.c | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-) (limited to 'src/os_mswin.c') diff --git a/src/os_mswin.c b/src/os_mswin.c index 80009533c6..096ac1bc5b 100644 --- a/src/os_mswin.c +++ b/src/os_mswin.c @@ -387,21 +387,13 @@ mch_FullName( int mch_isFullName(char_u *fname) { - // WinNT and later can use _MAX_PATH wide characters for a pathname, which - // means that the maximum pathname is _MAX_PATH * 3 bytes when 'enc' is - // UTF-8. - char szName[_MAX_PATH * 3 + 1]; - - // A name like "d:/foo" and "//server/share" is absolute - if ((fname[0] && fname[1] == ':' && (fname[2] == '/' || fname[2] == '\\')) - || (fname[0] == fname[1] && (fname[0] == '/' || fname[0] == '\\'))) - return TRUE; - - // A name that can't be made absolute probably isn't absolute. - if (mch_FullName(fname, (char_u *)szName, sizeof(szName) - 1, FALSE) == FAIL) - return FALSE; - - return pathcmp((const char *)fname, (const char *)szName, -1) == 0; + // A name like "d:/foo" and "//server/share" is absolute. "d:foo" is not. + // Another way to check is to use mch_FullName() and see if the result is + // the same as the name or mch_FullName() fails. However, this has quite a + // bit of overhead, so let's not do that. + return ((ASCII_ISALPHA(fname[0]) && fname[1] == ':' + && (fname[2] == '/' || fname[2] == '\\')) + || (fname[0] == fname[1] && (fname[0] == '/' || fname[0] == '\\'))); } /* -- cgit v1.2.3