[FFmpeg-devel] [PATCH] RTMP client support for lavf

[Kostya]

On Thu, Jul 30, 2009 at 10:20:46AM +0200, Diego Biurrun wrote:
> On Thu, Jul 30, 2009 at 09:16:22AM +0300, Kostya wrote:
> > 
> > --- libavformat/rtmpproto.c	(revision 0)
> > +++ libavformat/rtmpproto.c	(revision 0)
> > @@ -0,0 +1,677 @@
> > +/**
> > + * Puts HMAC-SHA2 digest of packet (except the place for digest itself) into that packet.
> 
> Hmmm

ok, made it clearer
 
> > +/**
> > + * Verifies that received server response has expected digest value.
> 
> that the, has the

the'd in this and some other places

> > +/**
> > + * Parses received packet and may perform some action depending on packet contents.
> 
> long line

broken

> > +                //extract a number from result
> 
> the result
> 
> > +/**
> > + * Opens RTMP connection and verifies that the stream can be played.
> > + *
> > + * URL syntax: rtmp://server[:port][/app][/playpath]
> > + *             where 'app' is first one or two directories in the path
> > + *             (e.g. /ondemand/, /flash/live/, etc.)
> > + *             and 'playpath' is a file name (the rest of the path,
> > + *             may be prefixed with "mp4:")
> 
> rtmp://streaming.somesite.com:4444/flash/live/mp4:/var/www/media/streamXXX.flv
> 
> ?

Yes, exactly in that manner, except it probably won't have extension at
all in that case.

> > +    s->max_packet_size = url_get_max_packet_size(rt->stream);
> > +    s->is_streamed = 1;
> 
> align

aligned

> > --- libavformat/rtmppkt.c	(revision 0)
> > +++ libavformat/rtmppkt.c	(revision 0)
> > @@ -0,0 +1,266 @@
> > +
> > +    if (url_read(h, &hdr, 1) != 1) {
> > +        return AVERROR(EIO);
> > +    }
> 
> pointless {}

removed
 
> Diego
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rtmp.patch
Type: text/x-diff
Size: 43164 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090730/a2230c8f/attachment.patch>