[FFmpeg-devel] libavcodec/exr : add support for datawindow who exceed display window
martin.vignali at gmail.com
Sat Apr 9 03:08:33 CEST 2016
2016-04-08 22:11 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
> On 4/7/16, Martin Vignali <martin.vignali at gmail.com> wrote:
> > Hello,
> > in attach, a patch with some modifications in size/data window
> > in order to manage
> > negative datawindow/display window, and datawindow who can exceed display
> > window
> > Size of the picture is now set from min and max value of the display
> > (before the patch, xmin, ymin of the display window are ignored (and
> > to be 0))
> > With the previous patch (fix channel detection), official samples
> > ./scanline/Blobbies.exr
> > can now be correctly decode.
> > DataWindow bigger than display window is a feature of the openExr.
> > It can be use in some softwares, (for example to improve blur quality)
> > Following the official documentation, for display, we need to use the
> > display window only.
> > In this patch, xmin, xmax, ymin, ymax, are now int32_t instead of
> > because these values can be negative. (See officiel sample
> > )
> > Comments welcome
> Crashes with:
> Strange, this sample didn't crash for me (Clang, OS 10.9). Decode it's
still not good after the patch but it's piz compression.
Can you tell me, what configure line, you use ?
> Please use int and unsigned instead of int32_t and uint32_t.
Where do you think it's better to use int or unsigned int instead of
int32_t, uint_32t ?
Exr Context ?
Decode block ?
Thanks for the comments.
More information about the ffmpeg-devel