Re: [FFmpeg-devel] [PATCH] avcodec/tiff: correct the default value of YCbCrSubsampling

2019-10-01 Thread Skakov Pavel
TIFF decoder uses wrong default YCbCrSubsampling value so it breaks on files that rely on standard default and omit the value. This patch broke fate-exif-image-tiff and fate-lavf-tiff. Patch updated, fixed one more error in code: rps check used subsampling even for non-subsampled formats.

Re: [FFmpeg-devel] [PATCH] avcodec/tiff: correct the default value of YCbCrSubsampling

2019-09-25 Thread James Almer
On 9/25/2019 5:48 AM, Carl Eugen Hoyos wrote: > Am Di., 24. Sept. 2019 um 20:32 Uhr schrieb Skakov Pavel : > >> TIFF decoder uses wrong default YCbCrSubsampling value so it breaks >> on files that rely on standard default and omit the value. > > Patch applied. > > Thank you, Carl Eugen This

Re: [FFmpeg-devel] [PATCH] avcodec/tiff: correct the default value of YCbCrSubsampling

2019-09-25 Thread Carl Eugen Hoyos
Am Di., 24. Sept. 2019 um 20:32 Uhr schrieb Skakov Pavel : > TIFF decoder uses wrong default YCbCrSubsampling value so it breaks > on files that rely on standard default and omit the value. Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] avcodec/tiff: correct the default value of YCbCrSubsampling

2019-09-24 Thread Carl Eugen Hoyos
Am Di., 24. Sept. 2019 um 20:32 Uhr schrieb Skakov Pavel : > TIFF decoder uses wrong default YCbCrSubsampling value so it breaks on > files that rely on standard default and omit the value. The patch seems to agree with the specification, can you provide a sample?

[FFmpeg-devel] [PATCH] avcodec/tiff: correct the default value of YCbCrSubsampling

2019-09-24 Thread Skakov Pavel
TIFF decoder uses wrong default YCbCrSubsampling value so it breaks on files that rely on standard default and omit the value. From 1b638417034ec212de4675db8e1695d1e5469d1d Mon Sep 17 00:00:00 2001 From: Pavel Skakov Date: Tue, 24 Sep 2019 20:30:59 +0300 Subject: [PATCH] avcodec/tiff: correct