[FFmpeg-devel] [PATCH] Publicly declare av_match_ext()

Michael Niedermayer michaelni
Sun Mar 28 01:31:50 CET 2010


On Fri, Mar 26, 2010 at 11:52:15PM +0100, Stefano Sabatini wrote:
> On date Saturday 2010-03-20 00:39:28 +0100, Stefano Sabatini encoded:
> > On date Friday 2010-03-12 00:36:27 +0100, Stefano Sabatini encoded:
> > > On date Wednesday 2010-03-10 14:15:48 -0500, Ronald S. Bultje encoded:
> > > > Hi,
> > > > 
> > > > On Wed, Mar 10, 2010 at 2:10 PM, Stefano Sabatini
> > > > <stefano.sabatini-lala at poste.it> wrote:
> > > > > On date Tuesday 2010-03-09 00:58:16 +0100, Stefano Sabatini encoded:
> > > > >> Hi, as in subject, fixes a warning in ffserver.c.
> > > > >
> > > > > Ping.
> > > > 
> > > > ffserver.c has several such problems.
> > > > 
> > > > Do we want FFmpeg (lavf/c/u) to contain such functionality? It seems
> > > > out-of-place to me. How about we simply add a (static) copy of the
> > > > match_ext() function in ffserver.c, with possibly a comment etc.?
> > > > 
> > > > I know, code duplication is bad, it's just a thought. Feel free to reject.
> > > 
> > > I don't have a strong idea and I recognize av_match_ext() is a little
> > > overspecialized to be used in a lib. Nonetheless I'd slightly prefer
> > > to make it public, at least it would avoid code duplication.
> > 
> > Waiting for the final decision of the maintainer...
> 
> Ping.

i dont maintain bikeshed.c

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

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- 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/20100328/cad4abb0/attachment.pgp>



More information about the ffmpeg-devel mailing list