You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I believe this is because getpath has hit its final fallback case of looking in ".\DLLs" and ".\Lib" for the standard library, and has not performed proper normalisation on these paths. So after removing the last segment, the result is "." which does not end in SEP.
So at least in this case, we can probably just ignore the assertion. However, I'm concerned that this would open up other cases where search paths may be added that are otherwise unintended.
The expected behaviour when the stdlib can't be found is a detailed error message, not a debug assertion/crash.
I left a message on the PR a day ago about a one-off error in the allocation of buff. The size should be MAXPATHLEN + 1. That's what's used everywhere else in PC/getpathp.c and what gets passed in the PathCchCombineEx(buff, MAXPATHLEN + 1, ...) call.