[FFmpeg-cvslog] r19516 - in trunk: Changelog libavformat/rtp_asf.c libavformat/rtsp.h

Diego Biurrun diego
Mon Jul 27 16:13:01 CEST 2009


On Mon, Jul 27, 2009 at 10:09:07AM -0400, Ronald S. Bultje wrote:
> 
> On Mon, Jul 27, 2009 at 10:07 AM, Diego Biurrun<diego at biurrun.de> wrote:
> > On Mon, Jul 27, 2009 at 04:00:10PM +0200, rbultje wrote:
> >>
> >> Log:
> >> Implement RTSP-MS/ASF packet parsing - this completes RTSP-MS support. See
> >> discussion in "[PATCH] RTSP-MS 14/15: ASF packet parsing" thread on mailinglist.
> >>
> >> --- trunk/libavformat/rtp_asf.c ? ? ? Mon Jul 27 15:01:44 2009 ? ? ? ?(r19515)
> >> +++ trunk/libavformat/rtp_asf.c ? ? ? Mon Jul 27 16:00:10 2009 ? ? ? ?(r19516)
> >> @@ -27,11 +27,73 @@
> >>
> >> +static int
> >> +rtp_asf_fix_header(uint8_t *buf, int len)
> >
> > ugly
> 
> You want them on a single line?

Yes please.  And no spaces between function name and opening '('.

> (I guess I always do it like this, I don't mind changing though.)

Let's try to maintain a uniform style across the project, i.e. K&R.

Diego



More information about the ffmpeg-cvslog mailing list