[MPlayer-cvslog] CVS: main/liba52 liba52_amd64_changes.diff, 1.1, 1.2

[Michael Niedermayer]

Hi

On Wed, Aug 10, 2005 at 10:59:13AM +0200, Reimar Döffinger wrote:
> Hi,
> On Wed, Aug 10, 2005 at 12:56:59AM +0200, Aurelien Jacobs CVS wrote:
> > CVS change done by Aurelien Jacobs CVS
> > 
> > Update of /cvsroot/mplayer/main/liba52
> > In directory mail:/var2/tmp/cvs-serv13732/liba52
> > 
> > Modified Files:
> > 	liba52_amd64_changes.diff 
> > Log Message:
> > sync liba52_amd64_changes.diff with latest fix
> > 
> > Index: liba52_amd64_changes.diff
> > ===================================================================
> > RCS file: /cvsroot/mplayer/main/liba52/liba52_amd64_changes.diff,v
> > retrieving revision 1.1
> > retrieving revision 1.2
> > diff -u -r1.1 -r1.2
> > --- liba52_amd64_changes.diff	5 Aug 2005 13:37:32 -0000	1.1
> > +++ liba52_amd64_changes.diff	9 Aug 2005 22:56:57 -0000	1.2
> > @@ -1182,7 +1182,7 @@
> >  +		"sub $16, %%"REG_D"			\n\t"
> >  +		"jnc 1b				 	\n\t"
> >  +		"pop %%"REG_BP"				\n\t"//no we didnt touch ebp *g*
> > -+		:: "r" (data), "r" (buf)
> > ++		:: "b" (data), "c" (buf)
> >  +		: "%"REG_S, "%"REG_D, "%"REG_a, "%"REG_d
> 
> Actually I think this should be necessary even withoug the AMD64 fixes

no, the amd64 patch changed bc -> rr (whoever did this needs to drink a lot
of cola, not to mention hiding such a change in the amd64 patch sucks)

[...]

-- 
Michael