Re: [AD] "clear" work bad with sub-bitmaps

[Javier González <xaviergonz@xxxxxxxxxx>]

Right fixed
thnx :)

Can somebody commit it?

----- Original Message -----
From: "Bob" <ohannessian@xxxxxxxxxx>
To: "Allegro Conductors" <conductors@xxxxxxxxxx>
Sent: Saturday, February 10, 2001 2:33 PM
Subject: Re: [AD] "clear" work bad with sub-bitmaps


>
>
> Javier González wrote:
> > can someone with good asm knowledge (i only know intel syntax ;) check
it?
>
> Fixed.
>
> --
> - Robert J Ohannessian
>
> "Microsoft code is probably O(n^20)" (My CS teacher)


----------------------------------------------------------------------------
----


> --- /allegro/src/i386/iblit8.s Sat Feb 10 08:20:20 2001
> +++ allegro/src/i386/iblit8.s Sat Feb 10 08:31:30 2001
> @@ -108,8 +108,7 @@
>
>     addl $8, %edi                 /* we set %edi to the next aligned
memory address */
>
> -   addl %eax, %ecx               /* and adjust %ecx to reflect the change
*/
> +   addl %eax, %ecx
>
>  clearMMX_aligned:
>     movl %ecx, %eax               /* save for later */
> @@ -195,7 +194,8 @@
>
>     addl $8, %edi                 /* we set %edi to the next aligned
memory address */
>
> -   addl %ecx, %eax               /* and adjust %ecx to reflect the change
*/
> +   addl %eax, %ecx
> +
>
>  clearMMXseg_aligned:
>     movl %ecx, %eax               /* save for later */
> --- /allegro/src/i386/iblit16.s Sun May 14 16:17:14 2000
> +++ allegro/src/i386/iblit16.s Sat Feb 10 08:28:38 2001
> @@ -109,7 +109,7 @@
>
>     addl $8, %edi                 /* we set %edi to the next aligned
memory address */
>
> -   addl %eax, %ecx               /* and adjust %ecx to reflect the change
*/
> +   addl %ecx, %eax               /* and adjust %ecx to reflect the change
*/
>
>  clearMMX_aligned:
>     movl %ecx, %eax               /* save for later */
> @@ -197,7 +197,7 @@
>
>     addl $8, %edi                 /* we set %edi to the next aligned
memory address */
>
> -   addl %ecx, %eax               /* and adjust %ecx to reflect the change
*/
> +   addl %eax, %ecx               /* and adjust %ecx to reflect the change
*/
>
>  clearMMXseg_aligned:
>     movl %ecx, %eax               /* save for later */
>