[FFmpeg-devel] [PATCH] lavfi: add avolume filter

Stefano Sabatini stefasab at gmail.com
Sat Nov 5 02:12:31 CET 2011


On date Friday 2011-11-04 17:15:25 +0100, Reimar Döffinger encoded:
> On Thu, Nov 03, 2011 at 10:14:30PM +0100, Reimar Döffinger wrote:
> > On Thu, Nov 03, 2011 at 01:24:14AM +0100, Stefano Sabatini wrote:
> > > +    .inputs  = (AVFilterPad[])  {{ .name           = "default",
> > > +                                   .type           = AVMEDIA_TYPE_AUDIO,
> > > +                                   .filter_samples = filter_samples,
> > > +                                   .min_perms      = AV_PERM_READ, AV_PERM_WRITE},
> > > +                                 { .name = NULL}},
> > 
> > Sorry for coming in with something unrelated and that probably was
> > discussed already a long time ago, but why are these not
> > "const AVFilterPad[]"?

Short answer: it was always like this. I guess the idea of having it
non-const was to be able to change pad type dynamically (e.g. setting
audio/video in a generic showinfo filter, and I'm not sure if this is
really doable) or maybe it was just a typo.

> Let me clarify that it looks to me that this causes _a lot_ of data
> to be in .data that should really be in .rodata (for performance,
> security and other reasons).

I think it should be possible to change the constness with no other
functional changes (but better to ifdef the change until the next
major bump, for avoiding to break compatibility).
-- 
FFmpeg = Frightening & Fiendish Mysterious Pacific Eretic Goblin


More information about the ffmpeg-devel mailing list