Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 07:27 PM, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- libavcodec/raw.c|1 + libavformat/nut.c |2 ++ tests/ref/fate/filter-pixfmts-copy |2 +- tests/ref/fate/filter-pixfmts-crop

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 08:35 PM, Mats Peterson wrote: On 02/09/2016 08:23 PM, Mats Peterson wrote: Wonderful, Michael. Mats But there is no provision for storing the proper palette yet, as far as I can see. Correct me if I'm wrong. And as far as I know, neither writing AVI or QuickTime will

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Michael Niedermayer
On Tue, Feb 09, 2016 at 08:35:07PM +0100, Mats Peterson wrote: > On 02/09/2016 08:23 PM, Mats Peterson wrote: > > >Wonderful, Michael. > > > >Mats > > But there is no provision for storing the proper palette yet, as far > as I can see. Correct me if I'm wrong. ive tried both converting to pal8

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 09:01 PM, Michael Niedermayer wrote: On Tue, Feb 09, 2016 at 08:35:07PM +0100, Mats Peterson wrote: On 02/09/2016 08:23 PM, Mats Peterson wrote: Wonderful, Michael. Mats But there is no provision for storing the proper palette yet, as far as I can see. Correct me if I'm

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 09:06 PM, Mats Peterson wrote: On 02/09/2016 09:03 PM, Mats Peterson wrote: I'm currently compiling here, so I can't tell yet... Anyway, AVI stores the palette in the output file, but only when using "-vcodec copy". QuickTime doesn't store any palette whatsoever. Just a

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 09:03 PM, Mats Peterson wrote: I'm currently compiling here, so I can't tell yet... Anyway, AVI stores the palette in the output file, but only when using "-vcodec copy". QuickTime doesn't store any palette whatsoever. Just a parenthesis. Yes, Michael, nut works just fine, and

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Michael Niedermayer
On Tue, Feb 09, 2016 at 09:09:18PM +0100, Mats Peterson wrote: > On 02/09/2016 09:06 PM, Mats Peterson wrote: > >On 02/09/2016 09:03 PM, Mats Peterson wrote: > >>I'm currently compiling here, so I can't tell yet... Anyway, AVI stores > >>the palette in the output file, but only when using "-vcodec

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 08:23 PM, Mats Peterson wrote: Wonderful, Michael. Mats But there is no provision for storing the proper palette yet, as far as I can see. Correct me if I'm wrong. Mats ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/raw.c|1 + libavformat/nut.c |2 ++ tests/ref/fate/filter-pixfmts-copy |2 +- tests/ref/fate/filter-pixfmts-crop |2 +- tests/ref/fate/filter-pixfmts-field |2 +-

Re: [FFmpeg-devel] [PATCH] nut: Add PAL8 support

2016-02-09 Thread Mats Peterson
On 02/09/2016 09:01 PM, Michael Niedermayer wrote: ive tried both converting to pal8 nut from yuv420p as well as storing rawvideo nut pal8 from a actual pal8 source, both worked Yes, it works fine. My bad. It seems nut is storing the palette at the end of the packet data rather than using a