<div class="gmail_quote">On Sun, Feb 15, 2009 at 1:45 PM, Johannes Schickel <span dir="ltr"><<a href="mailto:lordhoto@gmail.com">lordhoto@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Eugene Sandulenko schrieb:<br>
<div class="Ih2E3d">> My 2 cents is to extend this method a bit and add optional parameter to<br>
> it, that is the color value, defaulting to 0. And of course, reflect<br>
> that in the documentation.<br>
><br>
<br>
</div>Yeah I thought of that too, but I guess we should then rename it to<br>
something like OSystem::fillScreen.<br>
<font color="#888888"></font></blockquote><div><br></div></div>I was thinking along the same lines too.. there just seems something hackish about having it as an extra parameter, since it becomes just as efective for filling the screen with any given palette index rather than just clearing the screen.<br>
<br>If we did implement a separate fillScreen method, we could have the existing clearScreen method simply call fillScreen(0). That way engines that specifically need use a non-standard palette index can call fillScreen rather than clearScreen, but we keep the clearScreen() as is for the existing engines.<br>
<br>Paul.<br>