[FFmpeg-devel] RTP/SVQ3 payload parser

[Ronald S. Bultje]

Hi,

On Wed, Jul 29, 2009 at 12:22 PM, Diego Biurrun<diego at biurrun.de> wrote:
> On Wed, Jul 29, 2009 at 12:19:02PM -0400, Ronald S. Bultje wrote:
>> +#include <string.h>
>> +#include <libavcodec/get_bits.h>
>
> Use "" for local includes.

Actually, that one is unnecessary.

>> +static int sv3v_parse_sdp_line (AVFormatContext *s, int stream_index,
>> +
>> +static int sv3v_parse_packet (AVFormatContext *s, PayloadContext *sv,
>
> Please consistently avoid the space after the function name, i.e. K&R.

OK, done.

Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rtp-x_s3v_es.patch
Type: text/x-diff
Size: 7671 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090729/a5ff679f/attachment.patch>