This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author serhiy.storchaka
Recipients mark.dickinson, serhiy.storchaka, tim.peters
Date 2018-03-02.17:14:29
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1520010869.56.0.467229070634.issue32978@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2018-03-02 17:14:29serhiy.storchakasetrecipients: + serhiy.storchaka, tim.peters, mark.dickinson
2018-03-02 17:14:29serhiy.storchakasetmessageid: <1520010869.56.0.467229070634.issue32978@psf.upfronthosting.co.za>
2018-03-02 17:14:29serhiy.storchakalinkissue32978 messages
2018-03-02 17:14:29serhiy.storchakacreate