This is Stefan Stavrev's work, after I've squashed it and applied a few minor modifications on top of it. To wit:
* Lots of documenting comments. * Some minor formatting changes or other small changes for clarity in some places. * Move parallel_image to imagebufalgo.h and change slightly to remove the need for the R parameter entirely. I think that makes it even more generally useful. * Add testsuite/oiiotool-composite * Fix oiiotool default threads to correctly be hardware concurrency. I'm posting this review only to make it easy for Stefan and Chris Foster (and others, if interested) to look it over one more time, with both my and Stefan's changes together, before I do a final merge into the development master. Stefan's original pull request is here: https://github.com/OpenImageIO/oiio/pull/381 You can merge this Pull Request by running: git pull https://github.com/lgritz/oiio over Or you can view, comment on it, or merge it online at: https://github.com/OpenImageIO/oiio/pull/390 -- Commit Summary -- * ImageBufAlgo::over and oiiotool --over * More 'over' changes from LG atop Stefan's code: -- File Changes -- M src/CMakeLists.txt (2) M src/doc/oiiotool.tex (8) M src/include/imagebuf.h (85) M src/include/imagebufalgo.h (95) M src/libOpenImageIO/imagebuf.cpp (68) M src/libOpenImageIO/imagebufalgo.cpp (217) M src/libOpenImageIO/imageio.cpp (2) M src/oiiotool/oiiotool.cpp (32) A testsuite/oiiotool-composite/a.exr (0) A testsuite/oiiotool-composite/b.exr (0) A testsuite/oiiotool-composite/ref/a_over_b.exr (0) A testsuite/oiiotool-composite/ref/out.txt (2) A testsuite/oiiotool-composite/run.py (15) -- Patch Links -- https://github.com/OpenImageIO/oiio/pull/390.patch https://github.com/OpenImageIO/oiio/pull/390.diff --- Reply to this email directly or view it on GitHub: https://github.com/OpenImageIO/oiio/pull/390 _______________________________________________ Oiio-dev mailing list [email protected] http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
