Issue 2112: mmap.error should be a subclass of EnvironmentError and not a direct EnvironmentError

Issue2112

Created on 2008-02-14 14:57 by therve, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
mmap2.diff therve, 2008-02-14 14:57
mmap3.diff Chipaca, 2008-02-17 14:05 same as mmap2 but with NEWS entry
Messages (4)
msg62391 - (view) Author: Thomas Herve (therve) * Date: 2008-02-14 14:57
mmap.error in mmapmodule is a direct reference to
PyExc_EnvironmentError, whereas it should be a subclass. It makes it
provide a specific string representation, and allows to be more specific
when catching exceptions.

The attached patch tries to provide that.
msg62425 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2008-02-15 08:22
Sounds fine with me
msg62491 - (view) Author: John Rowland Lenton (Chipaca) * Date: 2008-02-17 14:05
Ran the regression test with this patch, all ok (on Ubuntu Gutsy).
Attached diff is the same, with the addition of the NEWS entry.
msg62500 - (view) Author: Facundo Batista (facundobatista) * (Python committer) Date: 2008-02-17 19:06
Fixed in r60878. Thanks everybody!
History
Date User Action Args
2022-04-11 14:56:30adminsetgithub: 46366
2008-02-17 19:06:19facundobatistasetstatus: open -> closed
resolution: fixed
messages: + msg62500
nosy: + facundobatista
2008-02-17 14:05:47Chipacasetfiles: + mmap3.diff
nosy: + Chipaca
messages: + msg62491
2008-02-15 08:22:37christian.heimessetpriority: normal
keywords: + patch, easy
messages: + msg62425
nosy: + christian.heimes
2008-02-14 14:57:50thervecreate