[FFmpeg-devel] [PATCH] ARM: update MpegEncContext offsets

Michael Niedermayer michaelni
Sat Jan 29 17:14:32 CET 2011


On Sat, Jan 29, 2011 at 04:59:34PM +0100, Luca Barbato wrote:
> On 01/29/2011 04:05 PM, Michael Niedermayer wrote:
> > On Sat, Jan 29, 2011 at 04:40:29AM +0000, Mans Rullgard wrote:
> >> ---
> >>  libavcodec/arm/asm-offsets.h |   12 ++++++------
> >>  1 files changed, 6 insertions(+), 6 deletions(-)
> > 
> > Id think you should say in the commit message why they are updated
> 
> Why is always known... This commit should reference the closest change
> in the struct maybe. Surely would be nicer have those folded with the
> commits changing the struct.

yes,

also if you guys want i can fix the code for this to become unneeded
by moving and sorting the affected fields as i previously explained

if mans doesnt want to implement my suggestion


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

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110129/22f7d6b6/attachment.pgp>



More information about the ffmpeg-devel mailing list