[Scummvm-devel] PORTERS: OSystem::clearScreen has been replaced by OSystem::fillScreen

[Marcus Comstedt]

Johannes Schickel <lordhoto at gmail.com> writes:

> DC related:
> I noticed that OSystem_Dreamcast derives from BaseBackend and still
> overwrites BaseBackend::fillScreen. This looks strange since
> OSystem_Dreamcast implements OSystem::lockScreen and
> OSystem::unlockScreen properly as far as I can tell it. Marcus might
> you check that out?

OSystem_Dreamcast::clearScreen() predates both OSystem::lockScreen()
and the default implementation of OSystem::clearScreen() (now
BaseBackend::fillScreen()) by almost two years, so I didn't override
this method because the default implementation was inappropriate,
but simply because it didn't exist at all.

As far as I can see, the implementation in BaseBackend should
work fine; the only difference it that it doesn't clear all the
allocated memory but only the top _screen_h lines, but nothing
below that should be significant anyway.


  // Marcus