bpo-37609: Add device path support in ntpath splitdrive#14841
bpo-37609: Add device path support in ntpath splitdrive#14841nsiregar wants to merge 5 commits intopython:mainfrom
Conversation
f69464e to
589e045
Compare
There was a problem hiding this comment.
Using the mod role here is not correct. Please see https://devguide.python.org/documenting/#id4.
There was a problem hiding this comment.
Thanks, I've updated it to use meth.
There was a problem hiding this comment.
Thanks, I've updated it
e92813b to
eb4e996
Compare
|
Sorry for not seeing this earlier, I have a few general suggestions:
|
| return p[:0], p | ||
|
|
||
|
|
||
| def is_unc_path(path, sep): |
There was a problem hiding this comment.
These functions are clearly private (e.g. parameterized by sep and colon), so the function names should have a leading underscore.
|
working on it |
|
For an alternate and more complete implementation, see the splitdrive.py file that I attached to bpo-37609. |
|
I managed to fix the issue elsewhere (in #91882), so I'm going to close this PR, but thank you very much for taking a look at it! :) |
Add device path support in
ntpath.splitdrivehttps://bugs.python.org/issue37609