Message158915
| Author |
neologix |
| Recipients |
neologix, phlogistonjohn, vinay.sajip |
| Date |
2012年04月21日.11:15:19 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1335006920.45.0.840355189671.issue14632@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Thanks for the report and patch.
I'm making Vinay nosy, as he's the logging expert.
A couple remarks:
- it would be nice to add your testcase to the logging tests (Lib/test/test_logging.py)
- I think you could probably save the whole fstat() result (instead of st_dev and st_inode): this would allow you to use os.pathsamestat() to check whether we're still refering to the same file, and it could maybe be useful someday (maybe use st_mtime)
- typo: "existance" -> "existence"
- I see you're catching OSError when calling fstat(): however, I don't see what error could be raised apart from EBADF (if the FD is closed, then it's likely a bug that shouldn't be masked, and it will blow up on the next write()), or I/O errors, that shouldn't be silenced |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2012年04月21日 11:15:20 | neologix | set | recipients:
+ neologix, vinay.sajip, phlogistonjohn |
| 2012年04月21日 11:15:20 | neologix | set | messageid: <1335006920.45.0.840355189671.issue14632@psf.upfronthosting.co.za> |
| 2012年04月21日 11:15:19 | neologix | link | issue14632 messages |
| 2012年04月21日 11:15:19 | neologix | create |
|