[FFmpeg-devel] [PATCH] RTSP-MS 10/15: ASF header parsing

[Michael Niedermayer]

On Wed, Feb 04, 2009 at 09:07:17AM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Wed, Feb 4, 2009 at 8:58 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Wed, Feb 04, 2009 at 08:35:10AM -0500, Ronald S. Bultje wrote:
> > [...]
> >> I know, it
> >> doesn't add comments, [...]
> > [...]
> >> +    RTSP_TRANSPORT_RDT,
> >> +    /**
> >> +     * This is not part of the public API and shouldn't be used outside ffmpeg.
> >> +     */
> >> +    RTSP_TRANSPORT_LAST
> > [...]
> >> -    RTSP_TRANSPORT_RDT,
> >> -    RTSP_TRANSPORT_LAST
> > [...]
> 
> Oh let me guess you don't want me to add that comment either? :-).

Its a sperate change -> seperate patch besodes i think RTSP_TRANSPORT_LAST
is a poor name when one sees it alone

and then one wonders why the enum is part of the public api at all?


patch below ok (in case luca doesnt reply)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090204/1c1a4d05/attachment.pgp>