Fix another round of squirrel coverity issues

[supertux.git] / src / video / texture.hpp

diff --git a/src/video/texture.hpp b/src/video/texture.hpp
index 0e4dd89..76964a9 100644 (file)
--- a/src/video/texture.hpp
+++ b/src/video/texture.hpp
@@ -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
@@ -14,14 +12,31 @@
 //  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.
+//  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+#ifndef HEADER_SUPERTUX_VIDEO_TEXTURE_HPP
+#define HEADER_SUPERTUX_VIDEO_TEXTURE_HPP
+
+#include <config.h>
 
-#ifndef __TEXTURE_HPP__
-#define __TEXTURE_HPP__
+#include <assert.h>
+#include <string>
+
+#include "supertux/globals.hpp"
+#include "video/texture_manager.hpp"
+
+/// bitset for drawing effects
+enum {
+  /** Don't apply anything */
+  NO_EFFECT = 0,
+  /** Draw the Surface upside down */
+  VERTICAL_FLIP = (1<<1),
+  /** Draw the Surface from left to down */
+  HORIZONTAL_FLIP = (1<<2),
+  NUM_EFFECTS
+};
 
-#include <SDL.h>
-#include <GL/gl.h>
+typedef unsigned int DrawingEffect;
 
 /**
  * This class is a wrapper around a texture handle. It stores the texture width
@@ -30,35 +45,32 @@
  */
 class Texture
 {
-protected:
+private:
   friend class TextureManager;
-  GLuint handle;
-  unsigned int width;
-  unsigned int height;
+  /* The name under which this texture is cached by the texture manager,
+   * or the empty string if not. */
+  std::string cache_filename;
 
 public:
-  Texture(unsigned int width, unsigned int height, GLenum glformat);
-  Texture(SDL_Surface* surface, GLenum glformat);
-  virtual ~Texture();
-  
-  GLuint get_handle() const
-  {
-    return handle;
-  }
-
-  unsigned int get_width() const
+  Texture() : cache_filename() {}
+  virtual ~Texture()
   {
-    return width;
+    if (TextureManager::current() && cache_filename != "")
+      /* The cache entry is now useless: its weak pointer to us has been
+       * cleared.  Remove the entry altogether to save memory. */
+      TextureManager::current()->reap_cache_entry(cache_filename);
   }
 
-  unsigned int get_height() const
-  {
-    return height;
-  }
+  virtual unsigned int get_texture_width() const = 0;
+  virtual unsigned int get_texture_height() const = 0;
+  virtual unsigned int get_image_width() const = 0;
+  virtual unsigned int get_image_height() const = 0;
 
 private:
-  void set_texture_params();
+  Texture(const Texture&);
+  Texture& operator=(const Texture&);
 };
 
 #endif
 
+/* EOF */