Steve Dower <[email protected]> added the comment:

I took a quick look at the patch and the main issue I see is the use of the 
MAX_PATH constant. We encourage people to disable this limit where possible, so 
using dynamic allocation where we need to reduce the path will be a better way 
to avoid this function breaking.

There should be other examples of this pattern in the file, but I don't think 
it'll matter here if we always copy the string. Better to avoid it, of course, 
but that complicates things.

----------

_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue32557>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to