Serhiy Storchaka <storchaka+cpyt...@gmail.com> added the comment:

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.

----------

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

Reply via email to