[Ffmpeg-devel] [PATCH] enable/disable (de)muxers
Måns Rullgård
mru
Mon Jul 10 01:22:20 CEST 2006
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.
--
M?ns Rullg?rd
mru at inprovide.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavf-rename-vars.patch
Type: text/x-patch
Size: 60049 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060710/242bbb6c/attachment.bin>
More information about the ffmpeg-devel
mailing list