Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-09 Thread Timothy Gu
On Sun, Nov 8, 2015 at 7:24 AM Kieran Kunhya wrote: > On 8 November 2015 at 00:30, Timothy Gu wrote: > > Allows one to do: > > > > ffmpeg -s 1920x1080 -i blah.v210 ... > > ffmpeg -s 1920x1080 -f v210x -i blah.yuv10 ... > > > > Fixes #1869. > > Looks good Thanks for reviewing;

Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-09 Thread Timothy Gu
On Mon, Nov 9, 2015 at 10:26 AM Calvin Walton wrote: > The ffmpeg command-line tool actually maps the "-s" and "-r" input > options to "video_size" and "framerate" respectively on the input > format. This will work automatically. > Thanks for the clarification :) Timothy ___

Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-09 Thread Calvin Walton
On Sat, 2015-11-07 at 22:22 -0300, James Almer wrote: > On 11/7/2015 9:45 PM, Timothy Gu wrote: > > +#define OFFSET(x) offsetof(V210DemuxerContext, x) > > +#define DEC AV_OPT_FLAG_DECODING_PARAM > > +static const AVOption v210_options[] = { > > +{ "video_size", "set frame size", OFFSET(width),

Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-08 Thread Kieran Kunhya
On 8 November 2015 at 00:30, Timothy Gu wrote: > Allows one to do: > > ffmpeg -s 1920x1080 -i blah.v210 ... > ffmpeg -s 1920x1080 -f v210x -i blah.yuv10 ... > > Fixes #1869. Looks good - I wonder if it can be merged with rawvideo though. Kieran __

Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-07 Thread James Almer
On 11/7/2015 11:46 PM, Timothy Gu wrote: > On Sat, Nov 07, 2015 at 10:22:34PM -0300, James Almer wrote: >> On 11/7/2015 9:45 PM, Timothy Gu wrote: >>> +#define OFFSET(x) offsetof(V210DemuxerContext, x) >>> +#define DEC AV_OPT_FLAG_DECODING_PARAM >>> +static const AVOption v210_options[] = { >>> +

Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-07 Thread Timothy Gu
On Sat, Nov 07, 2015 at 10:22:34PM -0300, James Almer wrote: > On 11/7/2015 9:45 PM, Timothy Gu wrote: > > +#define OFFSET(x) offsetof(V210DemuxerContext, x) > > +#define DEC AV_OPT_FLAG_DECODING_PARAM > > +static const AVOption v210_options[] = { > > +{ "video_size", "set frame size", OFFSET(w

Re: [FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-07 Thread James Almer
On 11/7/2015 9:45 PM, Timothy Gu wrote: > +#define OFFSET(x) offsetof(V210DemuxerContext, x) > +#define DEC AV_OPT_FLAG_DECODING_PARAM > +static const AVOption v210_options[] = { > +{ "video_size", "set frame size", OFFSET(width), AV_OPT_TYPE_IMAGE_SIZE, > {.str = NULL}, 0, 0, DEC }, Isn't th

[FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-07 Thread Timothy Gu
Allows one to do: ffmpeg -s 1920x1080 -i blah.v210 ... ffmpeg -s 1920x1080 -f v210x -i blah.yuv10 ... Fixes #1869. --- Remove .yuv from the list of extensions. --- libavformat/Makefile | 2 + libavformat/allformats.c | 2 + libavformat/v210.c | 133 ++

[FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-07 Thread Timothy Gu
Allows one to do: ffmpeg -s 1920x1080 -i blah.v210 ... ffmpeg -s 1920x1080 -f v210x -i blah.yuv10 ... Fixes #1869. --- libavformat/Makefile | 2 + libavformat/allformats.c | 2 + libavformat/v210.c | 133 +++ 3 files chan

[FFmpeg-devel] [PATCH] avformat: Add v210 demuxer

2015-11-07 Thread Timothy Gu
Allows one to do: ffmpeg -s 1920x1080 -i blah.v210 ... ffmpeg -s 1920x1080 -f v210x -i blah.yuv10 ... Fixes #1869. --- libavformat/Makefile | 2 + libavformat/allformats.c | 2 + libavformat/v210.c | 133 +++ 3 files chan