Message42772
Logged In: YES
user_id=21627
Clearing the error is bad, I agree. I see two options:
reraise the exception, deleting the result obtained so far
(i.e. as the code did that the latest patch removes), OR add
a byte string instead of the Unicode string into the result.
Even though I have proposed the latter in the past, I could
also accept the former; applications that anticipate that
exception then just need to re-invoke listdir with a byte
string, and deal with the result themselves.
With these changes, the patch is fine with me. |
|
Date |
User |
Action |
Args |
2007-08-23 15:20:30 | admin | link | issue683592 messages |
2007-08-23 15:20:30 | admin | create | |
|