[FFmpeg-devel] [PATCH 1/3] Move lpc_compute_autocorr() from DSPContext to a new struct LPCContext.
Fri Jan 21 04:29:21 CET 2011
On Fri, Jan 21, 2011 at 02:31:06AM +0000, M?ns Rullg?rd wrote:
> Luca Barbato <lu_zero at gentoo.org> writes:
> > On 01/21/2011 02:03 AM, Michael Niedermayer wrote:
> >> Dont you think that the code that contains these offsets would
> >> benefit more from cleanup to not break when random structs change
> >> than regularly waiting for failure and then find someone with an
> >> ARM build env to fix?
> > It had been discussed a bit in the past. There isn't a portable way to
> > get those offsets automatically given the field of use (arm is mostly
> > cross-built, more than often with exotic toolchain).
> A different approach would be to group the handful of field into a
> sub-struct and pass a pointer to that instead of the MpegEncContext
> pointer. However, given the general appearance of MpegEncContext,
> doing this can seem a bit scary.
simply moving the affected fields close together and passing a pointer to the
first seems easiest, it wont help the logic organization of the struct though
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel