Requirements at work caused us to need more robust handling of non-8-bit RLA files. In the process, I did a lot of simplification, refactoring, and commenting.
I'm very confident in the reader now. Even handles things like 10-bit RLAs embedded in 16 bit containers. I'm still testing the writer, I think there are a couple subtle bugs that I'm tracking down that leads to occasional LSB errors. I'll update when I have them fixed, but it's going to be very minor, so I think there's no reason not to start the review now for anyone who's interested. You can merge this Pull Request by running: git pull https://github.com/lgritz/oiio lg-rla Or you can view, comment on it, or merge it online at: https://github.com/OpenImageIO/oiio/pull/216 -- Commit Summary -- * Add a non-templated (general, but slower) version of bit_range_convert. * RLA overhaul: cleanup and lots of fixes for non-8-bit images. -- File Changes -- M src/doc/builtinplugins.tex (10) M src/include/fmath.h (14) M src/libOpenImageIO/fmath_test.cpp (12) M src/rla.imageio/rla_pvt.h (164) M src/rla.imageio/rlainput.cpp (548) M src/rla.imageio/rlaoutput.cpp (486) -- Patch Links -- https://github.com/OpenImageIO/oiio/pull/216.patch https://github.com/OpenImageIO/oiio/pull/216.diff --- Reply to this email directly or view it on GitHub: https://github.com/OpenImageIO/oiio/pull/216 _______________________________________________ Oiio-dev mailing list [email protected] http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
