[FFmpeg-devel] Fwd: [PATCH] Psygnosis YOP demuxer

Mohamed Naufal naufal11
Sat Dec 26 14:37:45 CET 2009


2009/12/26 Diego Biurrun <diego at biurrun.de>
[...]

> > --- Changelog (revision 20921)
> > +++ Changelog (working copy)
> > @@ -40,6 +40,7 @@
> >  - Core Audio Format demuxer
> >  - Atrac1 decoder
> >  - MD STUDIO audio demuxer
> > +- Psygnosis YOP demuxer
> >  - RF64 support in WAV demuxer
> >  - MPEG-4 Audio Lossless Coding (ALS) decoder
> >  - -formats option split into -formats, -codecs, -bsfs, and -protocols
>
> Add the new entry at the end.  Is the comment at the top of the file
> not clear enough?
>
> :">

[...]

>
> > +    video_stream->codec->extradata =
> > +        av_mallocz(video_stream->codec->extradata_size +
> > +                   FF_INPUT_BUFFER_PADDING_SIZE);
>
> ugly indentation
>
[...]

> > +    int palette_size = 4 + yop->num_pal_colors * 3;
> > +    int actual_video_data_size =
> > +        yop->frame_size - yop->sound_chunk_length - palette_size;
>
> ugly formatting again
>
> I apologize for making you repeat yourself.

[...]

Fixed.

Naufal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: yop_demux.patch
Type: text/x-patch
Size: 7685 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091226/80f97fde/attachment.bin>



More information about the ffmpeg-devel mailing list