On Fri, Oct 09, 2020 at 10:08:45AM +0100, Kieran O Leary wrote:
> Pinging as I think my comment got lost in the conversation, my main concern
> is about the Printing Density value aka 1,

This patch is for encoder, not for decoder (well decoder also need fixing but 
later).

The meaning of 1 value is same as 0 IMHO.

> 
> K
> 
> On Thu, Oct 8, 2020 at 10:25 AM Kieran O Leary <kieran.o.le...@gmail.com>
> wrote:
> 
> > Woah, more amazing film preservation patches, thank you!
> > From my uninformed reading of the code, does this only support the
> > detection of Linear, 709, 240M, 170M, Gamm22?  The reason I ask is that you
> > frequently see Printing Density appear as well, which has a value of  '1'
> > in the 801/802 offset..
> >
> > Best,
> >
> > Kieran O'Leary,
> > National Library of Ireland
> >
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to