Commit by coates :: r1283239 killbots/trunk/KDE/ (83 files in 5 dirs): (link)

Update my email address.

It turns out several headers were using an incorrect, non-existent
address, so it's a good thing I decided to do this.

Commit by yurchor :: r1280231 killbots/trunk/KDE/kdegames/doc/ (58 files in 5 dirs): (link)

Optimize images for documentation in PDF

Commit by trounev :: r1240899 killbots/trunk/KDE/kdegames/killbots/themes/ (mountainadventure.svgz robotkill.svgz): (link)

Removed the border around the background graphics.

Commit by coates :: r1175047 killbots/trunk/KDE/kdegames/killbots/view.cpp:

Make the Oxygen window gradient visible in unpainted areas of the scene.

Commit by coates :: r1164842 killbots/trunk/KDE/kdegames/killbots/killbots.kcfg:

Change the default theme back to Robot Kill.

Mummies are cool, but the name of the game *is* Killbots, not
Killmummies.

Commit by coates :: r1164843 killbots/trunk/KDE/kdegames/killbots/ (renderer.cpp renderer.h):

Remove caching of the theme aspect ratio and cursors from Renderer.

Caching the theme aspect ratio is no longer really neccessary now that
we have caching in KGameRenderer::boundsOnSprite().

When I first added the QCursor caching I did so just because I was
unsure of whether constructing cursors from existing pixmaps was
expensive or not. I just did some profiling and it seems that it is not
(on X11, at least).

Text color remains cached as it involves pixmap to image conversion and
is called by NumericDisplayItem::paint() which can occurs tens of times
per second.

Commit by coates :: r1164841 killbots/trunk/KDE/kdegames/killbots/mainwindow.cpp:

Do some minor code reorganisation.

Commit by coates :: r1164840 killbots/trunk/KDE/kdegames/killbots/ (5 files):

Use QGS::setBackgroundBrush instead of QGS::drawBackground.

This eliminates the need to worry about invalidating the view's
background cache. Although "rotating" the pixmap is slightly hackish,
overall I think this is a much cleaner proposal.

Commit by majewsky :: r1157587 killbots/trunk/KDE/kdegames/killbots/ (7 files):

Kill KGameRenderedPixmapItem class in favor of KGameRenderedItem.

Commit by coates :: r1156149 killbots/trunk/KDE/kdegames/killbots/ (sprite.cpp sprite.h):

Remove Sprite::setRenderSize() as KGRC::setRenderSize() isn't virtual.

Previously Sprite::setRenderSize() was just hiding KGameRendererClient
::setRenderSize(). While this worked fine in Killbots, it's still
hackish/ugly/wrong to pretend that polymorphism is present when it
actually isn't.

In this particular case, setting the offset inside receivePixmap works
perfectly well, but we might want to consider whether there is value in
making some of the methods in KGameRenderer and friends virtual to allow
for fancier reimplementations.

Commit by coates :: r1156144 killbots/trunk/KDE/kdegames/killbots/renderer.cpp:

Make Renderer's custom features load on demand.

Calculating this data at the end of setTheme proved to be too late, as
by that time the themeChanged signal has been emitted. Loading them on
demand just sidesteps that whole issue.

Commit by coates :: r1156148 killbots/trunk/KDE/kdegames/killbots/ (3 files):

Make NumericDisplayItem a KGameRenderedPixmapItem subclass.

Easy-cheesy and very straightforward.

Commit by coates :: r1156147 killbots/trunk/KDE/kdegames/killbots/ (scene.cpp sprite.cpp sprite.h):

Make Sprite a KGameRenderedPixmapItem subclass.

This change was particularly easy.

Commit by coates :: r1156146 killbots/trunk/KDE/kdegames/killbots/ (3 files):

Add a KGameRenderedPixmapItem class as a simpler alternative to KGRI.

While it has some fancy features, KGameRenderedItem's complexity can
also prove a hindrance. It doesn't provide access to the full QGPI
API. It adds the wait of QObject to every item (which is probably not a
big deal, but it bugs me.) One cannot override the implementation of
paint() because of the secret child item. Etc.

For the simple needs of Killbots all that is really needed is a
QGPI that can automatically fetch its pixmap from the KGR. This simple,
simple, simple class provides that.

Commit by coates :: r1156145 killbots/trunk/KDE/kdegames/killbots/ (mainwindow.cpp view.cpp view.h):

Invalidate the view background using the themeChanged signal.

More changes are likely here as I'm not happy with this.

Commit by coates :: r1156143 killbots/trunk/KDE/kdegames/killbots/ (10 files in 2 dirs):

Rename render.* to renderer.*.

Commit by coates :: r1156142 killbots/trunk/KDE/kdegames/killbots/ (6 files):

Replace custom Render namespace with a KGameRenderer derived singleton.

This is step one of the port to KGameRenderer. So far, so good.

Commit by coates :: r1138283 killbots/trunk/KDE/kdegames/killbots/ (mainwindow.cpp render.cpp render.h):

Destroy the K_GLOBAL_STATIC earlier to avoid QX11PixmapData warnings.

The K_GLOBAL_STATIC contains QCursors, which contain QPixmaps, so the
pixmaps end up being leaked as they're deleted after the QApplication
object is destroyed. This solution is a bit hackish, but meh.

Commit by coates :: r1138277 killbots/trunk/KDE/kdegames/ (5 files in 3 dirs):

Inherit from QGraphicsObject instead of QGraphicsItem and QObject.

Commit by coates :: r1046400 killbots/trunk/KDE/kdegames/killbots/ (7 files in 2 dirs):

Add a new ruleset parameter in order to make fastbots "patient".

When enable, fastbots will skip their extra turn to avoid colliding with
another robot. This can significantly increase their longevity and increase
their their threat.