C For C's Sake
C For C's Sake
You mean like in layer support using multi-dimensional bigarrays @bluddy?
I believe these are already covered by my gigantic unmerged GIF PR (perhaps modulo the write) which also changes various abstractions in order to support animated GIFs / progressive rendering,...
@rlepigre & @bluddy: How would you two feel about if I: - cherry-pick the relevant changes from @bluddy's present PR into the GIF PR - since the GIF PR is...
I haven't forgotten about this, having trouble upgrading to `decompress.1.1.0`, I will speak to @dinosaure about it.
If you have some concrete ideas we could add those to an `examples/` directory; I'd be in favor of that. For now, see the newly merged code: - [imagetool](https://github.com/rlepigre/ocaml-imagelib/blob/master/app/imagetool.ml#L105-L114) reads...
@bluddy I updated the README in response to your other report (#47). Do you think that it's addressed there? RE: conversion blowing up: It would a lot easier to track...
I think maybe just `raise (Corrupted_image "PNG palette index out of bounds")` would be sufficient? (wonder if this can be negative as well?)
I agree re: documenting exceptions, it would be nice if someone took the time to go through and document these. `result`: @rlepigre previously expressed that they are not so fond...
I think `of_bytes_raw` is nicer than `of_bytes_exn`, it make the calling code much prettier since you don't need to spray `String.(sub s 2 4)` everywhere. EDIT: To be clear I...
(this was not fixed in `2.9.0` / #13141 , I believe that was a typo in the issue ID there)