Merge pull request #548 from cchampet/fix_oiioPixelAspectRatioOfJpeg

Fix oiio pixel aspect ratio of jpeg

Author: cchampet

plugins/image/io/OpenImageIO/src/mainEntry.cpp

@@ -1,5 +1,5 @@
 #define OFXPLUGIN_VERSION_MAJOR 1
-#define OFXPLUGIN_VERSION_MINOR 2
+#define OFXPLUGIN_VERSION_MINOR 3
 
 #include <tuttle/plugin/Plugin.hpp>
 #include "reader/OpenImageIOReaderPluginFactory.hpp"

plugins/image/io/OpenImageIO/src/reader/OpenImageIOReaderPlugin.cpp

@@ -177,7 +177,7 @@ void OpenImageIOReaderPlugin::getClipPreferences(OFX::ClipPreferencesSetter& cli
         }
     }
 
-    float par = spec.get_float_attribute("PixelAspectRatio", 1.0f);
+    const float par = spec.get_float_attribute("PixelAspectRatio", 1.0f);
     clipPreferences.setPixelAspectRatio(*this->_clipDst, par);
     in->close();
 }

plugins/image/io/OpenImageIO/src/writer/OpenImageIOWriterProcess.tcc

@@ -547,8 +547,13 @@ void OpenImageIOWriterProcess<View>::writeImage(View& src, const std::string& fi
     spec.attribute("CompressionQuality", params._quality);
     spec.attribute("Orientation", params._orientation);
 
+    // write par, xdensity and ydensity to the output
+    // Some formats (ie. TIF, JPEG...) make the difference between those attributes.
+    // Some others (ie. DPX...) don't have density attributes and ignore them.
     const float par = _plugin._clipSrc->getPixelAspectRatio();
     spec.attribute("PixelAspectRatio", par);
+    spec.attribute("XResolution", par);
+    spec.attribute("YResolution", 1);
 
     if(!out->open(filepath, spec))
     {