Moved map drawing loop into the Map class and made it safer (it doesn't try to

draw tiles outside of the map area). The tmw.ini file was removed, we're using
config.xml now.

5 files changed, 59 insertions, 37 deletions

diff --git a/src/graphic/graphic.cpp b/src/graphic/graphic.cpp
index 0d8a926b..1602e13b 100644
--- a/src/graphic/graphic.cpp
+++ b/src/graphic/graphic.cpp
@@ -162,6 +162,16 @@ Graphics::~Graphics()
     _endDraw();
 }
 
+int Graphics::getWidth()
+{
+    return screen->w;
+}
+
+int Graphics::getHeight()
+{
+    return screen->h;
+}
+
 void Graphics::drawImageRect(
         int x, int y, int w, int h,
         Image *topLeft, Image *topRight,
@@ -367,8 +377,8 @@ void Engine::draw()
     map_x = (player_node->x - 13) * 32 + get_x_offset(player_node);
     map_y = (player_node->y -  9) * 32 + get_y_offset(player_node);
 
-    camera_x = map_x >> 5;
-    camera_y = map_y >> 5;
+    camera_x = map_x / 32;
+    camera_y = map_y / 32;
 
     int offset_x = map_x & 31;
     int offset_y = map_y & 31;
@@ -378,20 +388,11 @@ void Engine::draw()
     frame++;
 
     // Draw tiles below nodes
-    for (int j = 0; j < 20; j++) {
-        for (int i = 0; i < 26; i++) {
-            Image *tile0 = tiledMap->getTile(i + camera_x, j + camera_y, 0);
-            Image *tile1 = tiledMap->getTile(i + camera_x, j + camera_y, 1);
-
-            if (tile0) {
-                tile0->draw(screen, i * 32 - offset_x, j * 32 - offset_y);
-            }
-            if (tile1) {
-                tile1->draw(screen, i * 32 - offset_x, j * 32 - offset_y);
-            }
-        }
+    if (tiledMap) {
+        tiledMap->draw(guiGraphics, map_x, map_y, 0);
+        tiledMap->draw(guiGraphics, map_x, map_y, 1);
     }
-    
+
     // Draw items
     std::list<FloorItem*>::iterator floorItemIterator = floorItems.begin();
     while (floorItemIterator != floorItems.end())
@@ -531,21 +532,9 @@ void Engine::draw()
         // last drawed for fringe layer, draw the missing lines
     }
 
-    // Draw tiles above nodes
-    for (int j = 0; j < 20; j++) {
-        for (int i = 0; i < 26; i++) {
-            Image *tile = tiledMap->getTile(i + camera_x, j + camera_y, 2);
-
-            if (tile) {
-                tile->draw(screen, i * 32 - offset_x, j * 32 - offset_y);
-
-#ifdef DEBUG
-                guiGraphics->setColor(gcn::Color(0, 0, 0));
-                guiGraphics->drawRectangle(gcn::Rectangle(
-                            i * 32 - offset_x, j * 32 - offset_y, 32, 32));
-#endif
-            }
-        }
+    // Draw tiles below nodes
+    if (tiledMap) {
+        tiledMap->draw(guiGraphics, map_x, map_y, 2);
     }
 
     // Find a path from the player to the mouse, and draw it. This is for debug
diff --git a/src/graphic/graphic.h b/src/graphic/graphic.h
index 413c9d33..f8f7bb12 100644
--- a/src/graphic/graphic.h
+++ b/src/graphic/graphic.h
@@ -151,6 +151,16 @@ class Graphics : public gcn::SDLGraphics {
          */
         void updateScreen();
 
+        /**
+         * Returns the width of the screen.
+         */
+        int getWidth();
+
+        /**
+         * Returns the height of the screen.
+         */
+        int getHeight();
+
     private:
         Image *mouseCursor;
 };
diff --git a/src/map.cpp b/src/map.cpp
index 8b90df06..ede85d00 100644
--- a/src/map.cpp
+++ b/src/map.cpp
@@ -182,6 +182,30 @@ void Map::setSize(int width, int height)
     tiles = new Image*[width * height * 3];
 }
 
+void Map::draw(Graphics *graphics, int scrollX, int scrollY, int layer)
+{
+    int startX = scrollX / 32;
+    int startY = scrollY / 32;
+    int endX = (graphics->getWidth() + scrollX + 31) / 32;
+    int endY = (graphics->getHeight() + scrollY + 31) / 32;
+
+    if (startX < 0) startX = 0;
+    if (startY < 0) startY = 0;
+    if (endX >= width) endX = width - 1;
+    if (endY >= height) endY = height - 1;
+
+    for (int y = startY; y < endY; y++)
+    {
+        for (int x = startX; x < endX; x++)
+        {
+            Image *img = getTile(x, y, layer);
+            if (img) {
+                img->draw(screen, x * 32 - scrollX, y * 32 - scrollY);
+            }
+        }
+    }
+}
+
 void Map::setWalk(int x, int y, bool walkable)
 {
     metaTiles[x + y * width].walkable = walkable;
diff --git a/src/map.h b/src/map.h
index 2f03dcbe..056de6e1 100644
--- a/src/map.h
+++ b/src/map.h
@@ -25,6 +25,7 @@
 #define _TMW_MAP_H
 
 #include "being.h"
+#include "graphic/graphic.h"
 #include "resources/image.h"
 
 /**
@@ -97,6 +98,11 @@ class Map
         static Map *load(const std::string &mapFile);
 
         /**
+         * Draws the map to the given graphics output.
+         */
+        void draw(Graphics *graphics, int scrollX, int scrollY, int layer);
+
+        /**
          * Sets the size of the map. This will destroy any existing map data.
          */
         void setSize(int width, int height);
diff --git a/tmw.ini b/tmw.ini
deleted file mode 100644
index 4e7cc1f4..00000000
--- a/tmw.ini
+++ /dev/null
@@ -1,7 +0,0 @@
-host=animesites.de
-hwaccel=0
-port=6901
-remember=1
-screen=0
-sound=0
-username=