Data.Array.Base.elems needlessly calls bounds.
elems arr = case bounds arr of
(_l, _u) -> [unsafeAt arr i | i <- [0 .. numElements arr - 1]]It never uses the result.
I'd propose simplifying it to
elems arr = [unsafeAt arr i | i <- [0 .. numElements arr - 1]]It appears at some point someone optimized it to use the unsafeAt, but never removed the bounds check.
Trac metadata
| Trac field | Value |
|---|---|
| Version | 7.8.4 |
| Type | FeatureRequest |
| TypeOfFailure | OtherFailure |
| Priority | normal |
| Resolution | Unresolved |
| Component | Core Libraries |
| Test case | |
| Differential revisions | |
| BlockedBy | |
| Related | |
| Blocking | |
| CC | core-libraries-committee@haskell.org |
| Operating system | |
| Architecture |