Message313141
Thank you for your review Mark.
If left the OverflowError propagated I would catch it at the caller place (_read_float() is used only once) and reraise as aifc.Error. OverflowError is not expected exception. It never is raised for valid AIFC files, and the probability of raising it rather of aifc.Error for a random binary file is very small. I suppose that most users of this module don't catch it. See also issue32056.
But on other side, if there are files with an exponent of 0x7fff in wild, they are currently opened without errors. Raising an exception can be a regression. |
|
Date |
User |
Action |
Args |
2018-03-02 17:14:29 | serhiy.storchaka | set | recipients:
+ serhiy.storchaka, tim.peters, mark.dickinson |
2018-03-02 17:14:29 | serhiy.storchaka | set | messageid: <1520010869.56.0.467229070634.issue32978@psf.upfronthosting.co.za> |
2018-03-02 17:14:29 | serhiy.storchaka | link | issue32978 messages |
2018-03-02 17:14:29 | serhiy.storchaka | create | |
|