[Scummvm-devel] 0.9.0: Oops! Missing clipping in about dialog...
Marcus Comstedt
marcus at mc.pp.se
Thu Jun 22 12:38:54 CEST 2006
Johannes Schickel <lordhoto at gmail.com> writes:
> I just checked the code, but it should clip all stuff (check
> NewFont::drawChar, ThemeClassic::restoreBackground and
> ThemeClassic::addDirtyRect), also I used valgrind and I didn't get any
> warning. So maybe you should track it down and tell us the real problem.
> (I tried Classic theme in 320x200, which is used for Dreamcast as far as
> I saw it...)
Actually, I followed your advice and took a quick look at
NewFont::drawChar(), and I think I _can_ tell you the real problem.
The clipping code is there but it is all wrong. The code _draws_ at
ty + desc.ascent - bby - bbh + y
but checks
ty + y
against the screen size. So it can draw (desc.ascent-bby-bbh) pixels
outside the screen.
// Marcus
More information about the Scummvm-devel
mailing list