Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-05 Thread Carl Eugen Hoyos
2017-10-05 21:44 GMT+02:00 Carl Eugen Hoyos : > I tested the following with the sample from ticket #503: > $ ffmpeg -i rgba64le-lzw.tif -vf palettegen pal.png > $ ffmpeg -i rgba64le-lzw.tif -i pal.png -lavfi paletteuse out.gif > > The output file has its background (that is

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-05 Thread Carl Eugen Hoyos
2017-10-05 16:43 GMT+02:00 Bjorn Roche : > On Tue, Oct 3, 2017 at 4:02 AM, Carl Eugen Hoyos wrote: > >> 2017-10-03 1:52 GMT+02:00 Bjorn Roche : >> >> > Attached is a patch for paletteuse only. >> >> I tested the following with and without your

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-05 Thread Michael Niedermayer
On Thu, Oct 05, 2017 at 10:43:04AM -0400, Bjorn Roche wrote: > On Tue, Oct 3, 2017 at 4:02 AM, Carl Eugen Hoyos wrote: > > > 2017-10-03 1:52 GMT+02:00 Bjorn Roche : > > > > > Attached is a patch for paletteuse only. > > > > I tested the following with and

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-05 Thread Bjorn Roche
On Tue, Oct 3, 2017 at 4:02 AM, Carl Eugen Hoyos wrote: > 2017-10-03 1:52 GMT+02:00 Bjorn Roche : > > > Attached is a patch for paletteuse only. > > I tested the following with and without your patch: > $ ffmpeg -i fate-suite/lena.pnm -vf palettegen pal.png >

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-03 Thread Carl Eugen Hoyos
2017-10-03 1:52 GMT+02:00 Bjorn Roche : > Attached is a patch for paletteuse only. I tested the following with and without your patch: $ ffmpeg -i fate-suite/lena.pnm -vf palettegen pal.png $ ffmpeg -i fate-suite/lena.pnm -i pal.png -lavfi paletteuse out.png out.png changes

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-02 Thread Bjorn Roche
On Mon, Oct 2, 2017 at 4:19 PM, Carl Eugen Hoyos wrote: > 2017-10-02 20:44 GMT+02:00 Bjorn Roche : > > I submitted a complete patch separately > > No. > > What I wrote in my last email was (please > do not top-post here): > >> Instead please merge your

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-02 Thread Carl Eugen Hoyos
2017-10-02 20:44 GMT+02:00 Bjorn Roche : > I submitted a complete patch separately No. What I wrote in my last email was (please do not top-post here): >> Instead please merge your paletteuse patches >> into one patch, I believe the new patch should >> allow to encode a single

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-02 Thread Bjorn Roche
I submitted a complete patch separately ( http://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216793.html), but this is how git-email formatted things. How would you like me to proceed? On Mon, Oct 2, 2017 at 2:13 PM, Carl Eugen Hoyos wrote: > 2017-10-02 19:24 GMT+02:00

Re: [FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-02 Thread Carl Eugen Hoyos
2017-10-02 19:24 GMT+02:00 Bjorn Roche : > From: Bjorn Roche > -printf( "alpha 1, 2: %d, %d\n", c1[0], c2[0] ); Instead please merge your paletteuse patches into one patch, I believe the new patch should allow to encode a single RGBA frame with

[FFmpeg-devel] [PATCH 3/7] Removing some debugging

2017-10-02 Thread Bjorn Roche
From: Bjorn Roche --- libavfilter/vf_paletteuse.c | 65 - 1 file changed, 11 insertions(+), 54 deletions(-) diff --git a/libavfilter/vf_paletteuse.c b/libavfilter/vf_paletteuse.c index abee1b3735..21870c22f5 100644 ---