Re: [FFmpeg-devel] [PATCH 27/27] all: remove rest of YUVJ pixel formats usage
On Sun, Dec 10, 2017 at 11:26:58AM +0100, Paul B Mahol wrote: > On 12/10/17, Michael Niedermayer wrote: > > On Sat, Dec 09, 2017 at 04:38:19PM +0100, Paul B Mahol wrote: > >> Signed-off-by: Paul B Mahol > >> --- > >> fftools/ffmpeg_filter.c | 4 +-- > >> libavcodec/raw.c | 18 +++--- > >> libavcodec/utils.c | 11 -- > >> libavutil/pixdesc.c | 60 > >> > >> libavutil/pixfmt.h | 5 --- > >> libswscale/utils.c | 20 --- > >> tests/ref/fate/filter-pixdesc-yuvj411p | 1 - > >> tests/ref/fate/filter-pixdesc-yuvj420p | 1 - > >> tests/ref/fate/filter-pixdesc-yuvj422p | 1 - > >> tests/ref/fate/filter-pixdesc-yuvj440p | 1 - > >> tests/ref/fate/filter-pixdesc-yuvj444p | 1 - > >> tests/ref/fate/filter-pixfmts-copy | 5 --- > >> tests/ref/fate/filter-pixfmts-crop | 5 --- > >> tests/ref/fate/filter-pixfmts-field | 5 --- > >> tests/ref/fate/filter-pixfmts-fieldorder | 3 -- > >> tests/ref/fate/filter-pixfmts-hflip | 5 --- > >> tests/ref/fate/filter-pixfmts-il | 5 --- > >> tests/ref/fate/filter-pixfmts-null | 5 --- > >> tests/ref/fate/filter-pixfmts-pad| 5 --- > >> tests/ref/fate/filter-pixfmts-scale | 5 --- > >> tests/ref/fate/filter-pixfmts-swapuv | 5 --- > >> tests/ref/fate/filter-pixfmts-vflip | 5 --- > >> tests/ref/fate/sws-pixdesc-query | 15 > >> 23 files changed, 5 insertions(+), 186 deletions(-) > >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj411p > >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj420p > >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj422p > >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj440p > >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj444p > > > > I assume you already know but > > this breaks several jpeg related fate tests > > I tested fate with this and all before applied and fate passed fine. > > What you actually tested and how? What breaks for you? I tested with all patches applied in order, bisect pointed to this [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Modern terrorism, a quick summary: Need oil, start war with country that has oil, kill hundread thousand in war. Let country fall into chaos, be surprised about raise of fundamantalists. Drop more bombs, kill more people, be surprised about them taking revenge and drop even more bombs and strip your own citizens of their rights and freedoms. to be continued signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 27/27] all: remove rest of YUVJ pixel formats usage
On 12/10/17, Michael Niedermayer wrote: > On Sat, Dec 09, 2017 at 04:38:19PM +0100, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> fftools/ffmpeg_filter.c | 4 +-- >> libavcodec/raw.c | 18 +++--- >> libavcodec/utils.c | 11 -- >> libavutil/pixdesc.c | 60 >> >> libavutil/pixfmt.h | 5 --- >> libswscale/utils.c | 20 --- >> tests/ref/fate/filter-pixdesc-yuvj411p | 1 - >> tests/ref/fate/filter-pixdesc-yuvj420p | 1 - >> tests/ref/fate/filter-pixdesc-yuvj422p | 1 - >> tests/ref/fate/filter-pixdesc-yuvj440p | 1 - >> tests/ref/fate/filter-pixdesc-yuvj444p | 1 - >> tests/ref/fate/filter-pixfmts-copy | 5 --- >> tests/ref/fate/filter-pixfmts-crop | 5 --- >> tests/ref/fate/filter-pixfmts-field | 5 --- >> tests/ref/fate/filter-pixfmts-fieldorder | 3 -- >> tests/ref/fate/filter-pixfmts-hflip | 5 --- >> tests/ref/fate/filter-pixfmts-il | 5 --- >> tests/ref/fate/filter-pixfmts-null | 5 --- >> tests/ref/fate/filter-pixfmts-pad| 5 --- >> tests/ref/fate/filter-pixfmts-scale | 5 --- >> tests/ref/fate/filter-pixfmts-swapuv | 5 --- >> tests/ref/fate/filter-pixfmts-vflip | 5 --- >> tests/ref/fate/sws-pixdesc-query | 15 >> 23 files changed, 5 insertions(+), 186 deletions(-) >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj411p >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj420p >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj422p >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj440p >> delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj444p > > I assume you already know but > this breaks several jpeg related fate tests I tested fate with this and all before applied and fate passed fine. What you actually tested and how? What breaks for you? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 27/27] all: remove rest of YUVJ pixel formats usage
On Sat, Dec 09, 2017 at 04:38:19PM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > fftools/ffmpeg_filter.c | 4 +-- > libavcodec/raw.c | 18 +++--- > libavcodec/utils.c | 11 -- > libavutil/pixdesc.c | 60 > > libavutil/pixfmt.h | 5 --- > libswscale/utils.c | 20 --- > tests/ref/fate/filter-pixdesc-yuvj411p | 1 - > tests/ref/fate/filter-pixdesc-yuvj420p | 1 - > tests/ref/fate/filter-pixdesc-yuvj422p | 1 - > tests/ref/fate/filter-pixdesc-yuvj440p | 1 - > tests/ref/fate/filter-pixdesc-yuvj444p | 1 - > tests/ref/fate/filter-pixfmts-copy | 5 --- > tests/ref/fate/filter-pixfmts-crop | 5 --- > tests/ref/fate/filter-pixfmts-field | 5 --- > tests/ref/fate/filter-pixfmts-fieldorder | 3 -- > tests/ref/fate/filter-pixfmts-hflip | 5 --- > tests/ref/fate/filter-pixfmts-il | 5 --- > tests/ref/fate/filter-pixfmts-null | 5 --- > tests/ref/fate/filter-pixfmts-pad| 5 --- > tests/ref/fate/filter-pixfmts-scale | 5 --- > tests/ref/fate/filter-pixfmts-swapuv | 5 --- > tests/ref/fate/filter-pixfmts-vflip | 5 --- > tests/ref/fate/sws-pixdesc-query | 15 > 23 files changed, 5 insertions(+), 186 deletions(-) > delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj411p > delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj420p > delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj422p > delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj440p > delete mode 100644 tests/ref/fate/filter-pixdesc-yuvj444p I assume you already know but this breaks several jpeg related fate tests [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB it is not once nor twice but times without number that the same ideas make their appearance in the world. -- Aristotle signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel