Two small fixes, unrelated except that I noticed them both while using oiiotool.

1. By taking an early out of Oiiotool::read if the image is already read, avoid 
accidentally setting output_tilewidth and output_dataformat except for images 
as they are first read.  Doing this wrong was, by a convoluted path I do not 
wish to recount, leading to a situation where oiiotool outputs were becoming 
tiled when the inputs were not.  It's a long story, don't make me tell it.

2. OpenEXR, which I is one of the few (only?) formats that truly allows 
arbitrarily-named metadata , now suppresses format-specific metadata from 
non-OpenEXR formats.  What this means is that "oiiotool in.dpx -o out.exr" 
won't litter the OpenEXR output file with all manner of "dpx:foobar" 
DPX-specific metadata from the original input.


You can merge this Pull Request by running:

  git pull https://github.com/lgritz/oiio lg-oiiotool

Or you can view, comment on it, or merge it online at:

  https://github.com/OpenImageIO/oiio/pull/335

-- Commit Summary --

* oiiotool: only infer output tile size and data format for freshly-read files.
* OpenEXR: suppress format-specific metadata from other formats.

-- File Changes --

M src/oiiotool/oiiotool.cpp (6)
M src/openexr.imageio/exroutput.cpp (5)

-- Patch Links --

  https://github.com/OpenImageIO/oiio/pull/335.patch
  https://github.com/OpenImageIO/oiio/pull/335.diff

--- 
Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/pull/335
_______________________________________________
Oiio-dev mailing list
Oiio-dev@lists.openimageio.org
http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org

Reply via email to