Sprinter/mame
2
0
Fork 0
mirror of https://github.com/holub/mame synced 2025-05-22 13:48:55 +03:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

From: Christophe Jaillet [mailto:christophe.jaillet@wanadoo.fr]

Browse Source 
Sent: Sunday, March 29, 2009 4:12 AM
To: submit@mamedev.org
Subject: Fw: Another speed up in 'src\mame\video\mcatadv.c'

Hi, in the attached patch I sent on 16th February 2009 (see mail below), I
made a mistake.

The purpose of the patch was to reorder code in order to avoid useless
computation. It was *not* to modify the behaviour of the code.
The following lines were included in my patch at line 140. They should have
not !

-        destline[drawxpos] = (pix + (pen<<4));
+        destline[drawxpos] = (pix + pen);

So please, revert this part of the patch and put back the now missing
(pen<<4) in the code.

Games like 'nost' just look horrible without it !


Sorry for the mistake in the previous patch, it was a mix of two patches...


Best regards.
CJ
This commit is contained in:
Aaron Giles 2009-04-09 16:01:52 +00:00
parent 91f3cbf7ec
commit 40d01a0cdf
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/mame/video/mcatadv.c
View File

@ -132,13 +132,13 @@ static void draw_sprites( running_machine *machine, bitmap_t *bitmap, const rect
if((priline[drawxpos] < pri)) {
if (offset >= 0x500000*2) offset = 0;
pix = sprdata[offset/2];
if (offset & 1) pix = pix >> 4;
pix &= 0x0f;
if ((drawxpos >= cliprect->min_x) && (drawxpos <= cliprect->max_x) && pix)
destline[drawxpos] = (pix + (pen << 4));
}
destline[drawxpos] = (pix + pen);
}
offset++;
}
}