[FFmpeg-devel] [PATCH] swr: general doxy text about swr and example code.
michaelni at gmx.at
Tue Nov 20 21:13:09 CET 2012
On Tue, Nov 20, 2012 at 08:27:01PM +0100, Clément Bœsch wrote:
> On Tue, Nov 20, 2012 at 07:48:18PM +0100, Michael Niedermayer wrote:
> > > OTOH, some clarifications about the out_samples computation would be
> > > welcome; for example I'm curious in what case you would want to get a
> > > delay with a parameter different than the output sample rate (though, that
> > > is quiet unrelated to this example).
> > hmm, ill try to improve the swr_get_delay() doxy
> Thanks :)
> > or do you think it should be explained elsewhere ?
> I think it's fine in the swr_get_delay() doxy.
> > > > + * conversion context and everything associated with it must be freed with
> > > > + * swr_free().
> > > > + */
> > > >
> > > > #include <inttypes.h>
> > >
> > > unrelated: stdint.h is not enough?
> > My principle with headers is, add if compile fails, its not failing.
> > Iam quite sure though we could invest time to make the includes a
> > tighter subset which then will cause failures and the need to add more
> > inlcudes earlier on future changes ...
> > In the end this smells like work to add more work.
> In this particular case of a public header I think it's important to keep
> it clean for apps willing to do such "work". This header just needs
> [u]int*_t types for prototypes, it looks pretty safe to just include
feel free to change it
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel