Bug 560: Fix 1-pixel gaps between tiles near hidden areas.
[supertux.git] / src / object / tilemap.cpp
index a3e11ea..4a144e6 100644 (file)
@@ -1,12 +1,10 @@
-//  $Id$
-//
 //  SuperTux
 //  Copyright (C) 2006 Matthias Braun <matze@braunis.de>
 //
-//  This program is free software; you can redistribute it and/or
-//  modify it under the terms of the GNU General Public License
-//  as published by the Free Software Foundation; either version 2
-//  of the License, or (at your option) any later version.
+//  This program is free software: you can redistribute it and/or modify
+//  it under the terms of the GNU General Public License as published by
+//  the Free Software Foundation, either version 3 of the License, or
+//  (at your option) any later version.
 //
 //  This program is distributed in the hope that it will be useful,
 //  but WITHOUT ANY WARRANTY; without even the implied warranty of
 //  GNU General Public License for more details.
 //
 //  You should have received a copy of the GNU General Public License
-//  along with this program; if not, write to the Free Software
-//  Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
-
-#include <config.h>
+//  along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-#include <cassert>
-#include <algorithm>
-#include <iostream>
-#include <stdexcept>
 #include <math.h>
-#include <limits>
-
-#include "tilemap.hpp"
-#include "video/drawing_context.hpp"
-#include "level.hpp"
-#include "tile.hpp"
-#include "resources.hpp"
-#include "lisp/lisp.hpp"
-#include "lisp/list_iterator.hpp"
-#include "lisp/writer.hpp"
-#include "object_factory.hpp"
-#include "main.hpp"
-#include "log.hpp"
-#include "tile_set.hpp"
-#include "tile_manager.hpp"
-#include "scripting/tilemap.hpp"
-#include "scripting/squirrel_util.hpp"
 
-TileMap::TileMap(const TileSet *new_tileset)
-  : tileset(new_tileset), solid(false), speed_x(1), speed_y(1), width(0),
-    height(0), z_pos(0), x_offset(0), y_offset(0), movement(Vector(0,0)), drawing_effect(NO_EFFECT),
-    alpha(1.0), current_alpha(1.0), remaining_fade_time(0),
-    draw_target(DrawingContext::NORMAL)
+#include "object/tilemap.hpp"
+#include "scripting/squirrel_util.hpp"
+#include "scripting/tilemap.hpp"
+#include "supertux/globals.hpp"
+#include "supertux/object_factory.hpp"
+#include "supertux/tile_manager.hpp"
+#include "supertux/tile_set.hpp"
+#include "util/reader.hpp"
+
+TileMap::TileMap(const TileSet *new_tileset) :
+  tileset(new_tileset), 
+  tiles(),
+  solid(false), 
+  speed_x(1), 
+  speed_y(1), 
+  width(0),
+  height(0), 
+  z_pos(0), 
+  x_offset(0), 
+  y_offset(0), 
+  movement(0,0),
+  drawing_effect(NO_EFFECT),
+  alpha(1.0), 
+  current_alpha(1.0),
+  remaining_fade_time(0),
+  path(),
+  walker(),
+  draw_target(DrawingContext::NORMAL)
 {
 }
 
-TileMap::TileMap(const lisp::Lisp& reader)
-  : solid(false), speed_x(1), speed_y(1), width(-1),
-    height(-1), z_pos(0), x_offset(0), y_offset(0), movement(Vector(0,0)), drawing_effect(NO_EFFECT),
-    alpha(1.0), current_alpha(1.0), remaining_fade_time(0),
-    draw_target(DrawingContext::NORMAL)
+TileMap::TileMap(const Reader& reader) :
+  tileset(),
+  tiles(),
+  solid(false), 
+  speed_x(1), 
+  speed_y(1), 
+  width(-1),
+  height(-1), 
+  z_pos(0), 
+  x_offset(0),
+  y_offset(0),
+  movement(Vector(0,0)), 
+  drawing_effect(NO_EFFECT),
+  alpha(1.0), 
+  current_alpha(1.0), 
+  remaining_fade_time(0),
+  path(),
+  walker(),
+  draw_target(DrawingContext::NORMAL)
 {
   tileset = current_tileset;
   assert(tileset != NULL);
@@ -118,11 +129,25 @@ TileMap::TileMap(const lisp::Lisp& reader)
 }
 
 TileMap::TileMap(const TileSet *new_tileset, std::string name, int z_pos,
-                 bool solid, size_t width, size_t height)
-  : tileset(new_tileset), solid(solid), speed_x(1), speed_y(1), width(0),
-    height(0), z_pos(z_pos), x_offset(0), y_offset(0), movement(Vector(0,0)),
-    drawing_effect(NO_EFFECT), alpha(1.0), current_alpha(1.0),
-    remaining_fade_time(0), draw_target(DrawingContext::NORMAL)
+                 bool solid, size_t width, size_t height) :
+  tileset(new_tileset), 
+  tiles(),
+  solid(solid), 
+  speed_x(1), 
+  speed_y(1), 
+  width(0),
+  height(0), 
+  z_pos(z_pos), 
+  x_offset(0), 
+  y_offset(0), 
+  movement(Vector(0,0)),
+  drawing_effect(NO_EFFECT), 
+  alpha(1.0), 
+  current_alpha(1.0),
+  remaining_fade_time(0), 
+  path(),
+  walker(),
+  draw_target(DrawingContext::NORMAL)
 {
   this->name = name;
 
@@ -134,23 +159,6 @@ TileMap::~TileMap()
 }
 
 void
-TileMap::write(lisp::Writer& writer)
-{
-  writer.start_list("tilemap");
-
-  writer.write("z-pos", z_pos);
-
-  writer.write("solid", solid);
-  writer.write("speed", speed_x);
-  writer.write("speed-y", speed_y);
-  writer.write("width", width);
-  writer.write("height", height);
-  writer.write("tiles", tiles);
-
-  writer.end_list("tilemap");
-}
-
-void
 TileMap::update(float elapsed_time)
 {
   // handle tilemap fading
@@ -180,42 +188,55 @@ TileMap::update(float elapsed_time)
 void
 TileMap::draw(DrawingContext& context)
 {
-  // skip draw if current opacity is set to 0.0
+  // skip draw if current opacity is 0.0
   if (current_alpha == 0.0) return;
 
   context.push_transform();
-  context.push_target();
-  context.set_target(draw_target);
+  if(draw_target != DrawingContext::NORMAL) {
+    context.push_target();
+    context.set_target(draw_target);
+  }
 
   if(drawing_effect != 0) context.set_drawing_effect(drawing_effect);
   if(current_alpha != 1.0) context.set_alpha(current_alpha);
 
+  /* Force the translation to be an integer so that the tiles appear sharper.
+   * For consistency (i.e., to avoid 1-pixel gaps), this needs to be done even
+   * for solid tilemaps that are guaranteed to have speed 1.
+   * FIXME Force integer translation for all graphics, not just tilemaps. */
   float trans_x = roundf(context.get_translation().x);
   float trans_y = roundf(context.get_translation().y);
   context.set_translation(Vector(int(trans_x * speed_x),
                                  int(trans_y * speed_y)));
 
-  /** if we don't round here, we'll have a 1 pixel gap on screen sometimes.
-   * I have no idea why */
-  float start_x = int((roundf(context.get_translation().x) - roundf(x_offset)) / 32) * 32 + roundf(x_offset);
-  float start_y = int((roundf(context.get_translation().y) - roundf(y_offset)) / 32) * 32 + roundf(y_offset);
-  float end_x = std::min(start_x + SCREEN_WIDTH + 32, float(width * 32 + roundf(x_offset)));
-  float end_y = std::min(start_y + SCREEN_HEIGHT + 32, float(height * 32 + roundf(y_offset)));
-  int tsx = int((start_x - roundf(x_offset)) / 32); // tilestartindex x
-  int tsy = int((start_y - roundf(y_offset)) / 32); // tilestartindex y
+  int tsx = int((context.get_translation().x - x_offset) / 32); // tilestartindex x
+  int tsy = int((context.get_translation().y - y_offset) / 32); // tilestartindex y
+  tsx = std::max(tsx, 0);
+  tsy = std::max(tsy, 0);
+  float start_x = tsx * 32 + x_offset;
+  float start_y = tsy * 32 + y_offset;
+  float end_x = start_x + SCREEN_WIDTH + 32;
+  float end_y = start_y + SCREEN_HEIGHT + 32;
 
   Vector pos;
   int tx, ty;
-  for(pos.x = start_x, tx = tsx; pos.x < end_x; pos.x += 32, ++tx) {
-    for(pos.y = start_y, ty = tsy; pos.y < end_y; pos.y += 32, ++ty) {
-      if ((tx < 0) || (ty < 0)) continue;
-      const Tile* tile = tileset->get(tiles[ty*width + tx]);
+
+  for(pos.x = start_x, tx = tsx; (pos.x < end_x) && (tx < width); pos.x += 32, ++tx) {
+    for(pos.y = start_y, ty = tsy; (pos.y < end_y) && (ty < height); pos.y += 32, ++ty) {
+      int index = ty*width + tx;
+      assert (index >= 0);
+      assert (index < (width * height));
+
+      if (tiles[index] == 0) continue;
+      const Tile* tile = tileset->get(tiles[index]);
       assert(tile != 0);
       tile->draw(context, pos, z_pos);
-    }
-  }
+    } /* for (pos y) */
+  } /* for (pos x) */
 
-  context.pop_target();
+  if(draw_target != DrawingContext::NORMAL) {
+    context.pop_target();
+  }
   context.pop_transform();
 }
 
@@ -244,20 +265,20 @@ void
 TileMap::expose(HSQUIRRELVM vm, SQInteger table_idx)
 {
   if (name.empty()) return;
-  Scripting::TileMap* interface = new Scripting::TileMap(this);
-  expose_object(vm, table_idx, interface, name, true);
+  scripting::TileMap* _this = new scripting::TileMap(this);
+  expose_object(vm, table_idx, _this, name, true);
 }
 
 void
 TileMap::unexpose(HSQUIRRELVM vm, SQInteger table_idx)
 {
   if (name.empty()) return;
-  Scripting::unexpose_object(vm, table_idx, name);
+  scripting::unexpose_object(vm, table_idx, name);
 }
 
 void
 TileMap::set(int newwidth, int newheight, const std::vector<unsigned int>&newt,
-    int new_z_pos, bool newsolid)
+             int new_z_pos, bool newsolid)
 {
   if(int(newt.size()) != newwidth * newheight)
     throw std::runtime_error("Wrong tilecount count.");
@@ -325,7 +346,6 @@ TileMap::get_tile_id(int x, int y) const
   return tiles[y*width + x];
 }
 
-
 const Tile*
 TileMap::get_tile(int x, int y) const
 {
@@ -379,7 +399,6 @@ TileMap::fade(float alpha, float seconds)
   this->remaining_fade_time = seconds;
 }
 
-
 void 
 TileMap::set_alpha(float alpha)
 {
@@ -395,5 +414,6 @@ TileMap::get_alpha()
 {
   return this->current_alpha;
 }
-  
-IMPLEMENT_FACTORY(TileMap, "tilemap");
+
+/* EOF */