On Sat, Dec 22, 2018 at 09:28:47PM +0100, Steinar H. Gunderson wrote: > On Sat, Dec 22, 2018 at 09:18:11PM +0100, Paul B Mahol wrote: > > Unacceptable, I'm not adding another yuv420p variant. > > Well, if returning YCC as YCC is unacceptable, and converting YCC to RGB is > unacceptable, I believe your only choices are: > > 1. Try to convert YCC to Y'CbCr ignoring the gamma curve, which will return > images that look wrong and cannot be repaired by reasonable means. > 2. Return YCC mislabeled as something else, which will look even more wrong. > 3. Don't include PhotoCD support in FFmpeg.
a user switch could be added to choose between RGB and incorrect YCbCr the alternative would be adding support to swscale for the new variant and some means to transport the information to it so its converted correctly. for example by using AVColor* enums I think doing this in swscale is the "proper" solution but doing it in the decoder is not that unreasonable as this is a rather odd format. Also another pixfmt is not ideal as paul already mentioned. Though i would not outright reject that option, its probably easier to do than using AVColor* enums and have this all working well either way, iam fine with whatever others are fine with ... Thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I know you won't believe me, but the highest form of Human Excellence is to question oneself and others. -- Socrates
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
