Mark Mc Mahon <mtnbikingm...@gmail.com> added the comment:

For Directory.make_short() the only things which are left could be considered 
splitting hairs.

1. Do we need to remove '"\/:' from the file name - if these characters are 
there then the file name is not valid in the first place.

2. These characters '+,;=[]' should really be changed to '_' if we want to 
match more closely how Windows makes short names.

I can write a patch for the above if they should be fixed and either way I can 
add tests for this function. (the test I had for testing my own patch - almost 
completely pass - but the only failure was depending on point 2.)

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue7639>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to