On 12/22/11, Diego Biurrun <di...@biurrun.de> wrote:
> On Thu, Dec 22, 2011 at 06:10:27PM +0000, Paul B Mahol wrote:
>> Signed-off-by: Paul B Mahol <one...@gmail.com>
>> ---
>>  Changelog                |    1 +
>>  doc/general.texi         |    1 +
>>  libavformat/Makefile     |    1 +
>>  libavformat/allformats.c |    1 +
>>  libavformat/smjpeg.c     |  183
>> ++++++++++++++++++++++++++++++++++++++++++++++
>>  libavformat/version.h    |    2 +-
>>  6 files changed, 188 insertions(+), 1 deletions(-)
>>  create mode 100644 libavformat/smjpeg.c
>
> Did you verify this compiles standalone, i.e. with
>
>   configure --disable-everything --enable-demuxer=smjpeg

Yes I did.
>
>> --- /dev/null
>> +++ b/libavformat/smjpeg.c
>> @@ -0,0 +1,183 @@
>> +static int smjpeg_read_header(AVFormatContext *s, AVFormatParameters *ap)
>> +{
>> +    SMJPEGContext *sc = s->priv_data;
>> +    AVStream *ast = NULL, *vst = NULL;
>> +    AVIOContext *pb = s->pb;
>> +    uint32_t version, htype, hlength, duration;
>> +    char *comment;
>> +
>> +    avio_skip(pb, 8); // magic
>> +    version = avio_rb32(pb);
>> +    if (version) {
>> +        av_log_ask_for_sample(s, "unknown version %d\n", version);
>> +    }
>
> nit: drop the braces
Done.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to