[FFmpeg-devel] [PATCH 3/3] lavfi: fix doxygen doc for AVFilterLink.time_base

[Stefano Sabatini]

On date Tuesday 2011-08-09 16:04:59 +0200, Michael Niedermayer encoded:
> On Mon, Aug 08, 2011 at 11:59:48AM +0200, Stefano Sabatini wrote:
> > On date Saturday 2011-08-06 20:26:01 +0200, Michael Niedermayer encoded:
> > > On Sat, Aug 06, 2011 at 01:29:18PM +0200, Stefano Sabatini wrote:
> > [...]
> > > > What about removing sample_rate from link, and use time_base instead:
> > > >    if (audio)
> > > >        time_base = (AVRational) { 1, sample_rate };
> > > > ?
> > > 
> > > what would you do when the input to the filter chain uses a timebase
> > > that has units that arent representable in the sample rate ?
> > 
> > So are you suggesting to drop sample_rate and use time_base instead?
> > What else? Please be more explicit.
> 
> i would suggest that time_base be used, yes

Check patch.
-- 
FFmpeg = Freak Funny Muttering Ponderous Evangelical Governor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0007-lavfi-drop-AVFilterLink.sample_rate-at-the-next-majo.patch
Type: text/x-diff
Size: 8065 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110911/e99e4ba3/attachment.bin>