[Ffmpeg-devel] [PATCH] enable/disable (de)muxers

Diego Biurrun diego
Mon Jul 10 03:53:37 CEST 2006


On Mon, Jul 10, 2006 at 02:38:26AM +0100, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Mon, Jul 10, 2006 at 12:22:20AM +0100, M?ns Rullg?rd wrote:
> >> Diego Biurrun <diego at biurrun.de> writes:
> >> 
> >> > On Sun, Jul 09, 2006 at 11:35:21PM +0100, M?ns Rullg?rd wrote:
> >> >> Diego Biurrun <diego at biurrun.de> writes:
> >> >> 
> >> >> > On Sun, Jul 09, 2006 at 11:10:40PM +0100, M?ns Rullg?rd wrote:
> >> >> >> M?ns Rullg?rd <mru at inprovide.com> writes:
> >> >> >> 
> >> >> >> > OK, here's the initial patch for review.  I'd like to have someone
> >> >> >> > else cast an eye on it before I commit, just in case I did something
> >> >> >> > really silly.
> >> >> >> 
> >> >> >> Hmm... already a flaw found.  Updated patch attached.
> >> >> >> 
> >> >> >> --- ffplay.c	(revision 5695)
> >> >> >> +++ ffplay.c	(working copy)
> >> >> >> @@ -1790,7 +1790,7 @@
> >> >> >>  #ifdef CONFIG_NETWORK
> >> >> >> -    use_play = (ic->iformat == &rtsp_demux);
> >> >> >> +    use_play = (ic->iformat == &rtsp_demuxer);
> >> >> >>  #else
> >> >> >> @@ -1884,7 +1884,7 @@
> >> >> >>          }
> >> >> >> -        if (is->paused && ic->iformat == &rtsp_demux) {
> >> >> >> +        if (is->paused && ic->iformat == &rtsp_demuxer) {
> >> >> >>              /* wait 10 ms to avoid trying to get another packet */
> >> >> >
> >> >> > Here and below I'd prefer to see all those (semi-cosmetic) renaming
> >> >> > operations done in a separate patch/commit, this will shrink this patch
> >> >> > considerably and make it easier to review.
> >> >> 
> >> >> I suppose I could first do all the renames, keeping the structure.
> >> >> Such a patch would have the same size as this one (more or less).
> >> >> That would then be followed by another patch to remove the 'static' in
> >> >> the definitions, making for another patch of about the same size.
> >> >
> >> > I was expecting it to be considerably smaller, after a closer reading of
> >> > the patch I may be wrong and the saving might not be so big after all..
> >> 
> >> Oh well, here it is.  Just the renamings of AVInput/OutputFormat vars.
> >
> > Looks good and was easy to review, commit.
> 
> And here goes part 2.

Looks good at a glance.

Diego




More information about the ffmpeg-devel mailing list