[FFmpeg-devel] [PATCH] RTSP muxer, round 5

Ronald S. Bultje rsbultje
Mon Feb 22 23:35:38 CET 2010


Hi,

On Mon, Feb 22, 2010 at 4:31 PM, Martin Storsj? <martin at martin.st> wrote:
> On Mon, 22 Feb 2010, Ronald S. Bultje wrote:
>> On Mon, Feb 22, 2010 at 4:01 PM, Martin Storsj? <martin at martin.st> wrote:
>> > Updated patch attached.
>>
>> + ? ?if (reply->status_code != RTSP_STATUS_OK) {
>> + ? ? ? ?return -1;
>> + ? ?}
>>
>> I'd say remove the brackets here. With that, patch OK. Congratulations!
>
> Applied with this change.

I think the attached patch fixes a compilation issue in mplayer with
librtsp enabled. It removes prototypes of rtsp_pause/resume()
(non-existing?) and it also prefixes all public rtsp api with ff_,
which I forgot in review (and Diego kindly reminded me).

Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rename-rtsp-public-functions.patch
Type: text/x-diff
Size: 15812 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100222/37bd6cd5/attachment.patch>



More information about the ffmpeg-devel mailing list