Image2

[jeffreyrosenbluth]

I think this is simpler and better. You can ignore the comments in the pull request for image.
I kept uncheckedImageRef since we need to cache the width and height in loadImageExt anyway so that backends can use them, but the user does not need to provide them explicitly when using the load functions.

There is a companion rasterific branch, but the original cairo image branch works as is.

[jeffreyrosenbluth]

Here is how you would use the rasterific api.
From KnightTour:

 main = do
    d <- loadImageEmb "white-knight.png"
    let img = case d of
               Left e -> error $ show e
               Right d' -> image d' # scaleUToX 1 
   mainWith $ example img

and in cairo

 main = do
    d <- loadImageExt "white-knight.png"
    let img = case d of
               Left e -> error $ show e
               Right d' -> image d' # scaleUToX 1
    mainWith $ example img # pad 1.1

[byorgey]

Yes, I like this better too.

[jeffreyrosenbluth]

Great, I'm inclined to merge and revisit images after fixpoint.
Mainly becuase I want to release a fully featured rasterific backend,
which depends on image2

On Wed, Apr 9, 2014 at 10:27 AM, Brent Yorgey notifications@github.comwrote:

Yes, I like this better too.

Reply to this email directly or view it on GitHubhttps://github.com//pull/174#issuecomment-39970081
.

[byorgey]

I merged because it wasn't worth holding it up over more bikeshedding. But what do you think of changing the name of fromRaster to rasterDia?

[jeffreyrosenbluth]

Probably and improvement.