[MPlayer-dev-eng] [PATCH] New native RTSP demuxer
Benjamin Zores
ben at geexbox.org
Sat Jun 17 15:00:00 CEST 2006
On Sat, 17 Jun 2006 14:50:39 +0200
Roberto Togni <r_togni at tiscali.it> wrote:
> > -const char rtsp_protocol_version[]="RTSP/1.0";
> > +#define RTSP_PROTOCOL_VERSION "RTSP/1.0"
> >
>
> Why? Isn't it better to move all the text commands (RTSP_) as
> static const instead of #define them?
the defines are here because i'll use them in further patch for rtsp_session.c file.
So it can't be static.
> Please remove all the sizeof(char), it's useless and misleading when
> auditing memory allocations.
ok i'll do so.
> > +char *rtsp_get_host (rtsp_t * s)
> > +{
> > + return s->host;
> > +}
> > +
>
> Ok if it's going to be replaced with something more complex in your
> later patches, else I don't think we need a function to get a field
> from a struct :)
fine by me, i just wanted to have the same prototypes than for the 2 previous functions:
char *rtsp_get_session(rtsp_t *s) {
return s->session;
}
char *rtsp_get_mrl(rtsp_t *s) {
return s->mrl;
}
but i'm ok with not adding bloat.
> I still haven't tested the patch, but if it works you can commit it.
no i can't :-)
Ben
--
"My life, and by extension everyone else's is meaningless."
Bender, Futurama
More information about the MPlayer-dev-eng
mailing list