[FFmpeg-cvslog] r12504 - in trunk/libavformat: rtsp.c rtsp.h

[Ronald S. Bultje]

Hi,

On Wed, Mar 19, 2008 at 10:08 AM, Luca Barbato <lu_zero at gentoo.org> wrote:

> rbultje wrote:
> > Author: rbultje
> > Date: Wed Mar 19 15:05:08 2008
> > New Revision: 12504
> >
> > Log:
> > Allow cycling between different protocols (TCP, UDP or multicast) so
> that if
> > one doesn't work, we can try the next one (i.e. trial-error protocol
> auto-
> > probing).
> >
> > Discussed and approved in "[PATCH] RTSP alternate protocol 2-3/3".
>
>
> > +        if (reply->status_code == 461 /* Unsupported protocol */ && i
> == 0) {
>
> What about having the rtsp/http codes in macros?


(They're not exactly the same for rtsp/http.) It's a very interesting idea,
and I've played with the slightly related idea of including tables with
macros and text strings for each of those codes. This would mean being able
to generate useful error messages for display to users. Although I haven't
done a complete implementation, I found it very comforting to see a "url not
found" (or whatever) error rather than just a "-1" return value... This is
all related to my somewhat forgotten errno (and ipv6) patches that I should
resurrect at some point...

My current code is mostly client-side (i.e. in the app using lav*), though,
I'll port it to lav* if there's consensus that that's the right place for
it.

Ronald