From b05920748f2eb83839c7925f3c83530b733331f1 Mon Sep 17 00:00:00 2001
From: Alexander Sulfrian <alexander@sulfrian.net>
Date: Mon, 17 Sep 2012 01:03:35 +0200
Subject: add const where applicable

---
 src/menu/control.cpp | 10 +++++-----
 src/menu/control.hpp | 16 ++++++++++++----
 src/utils/image.cpp  |  4 ++--
 src/utils/image.hpp  |  6 +++---
 4 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/src/menu/control.cpp b/src/menu/control.cpp
index 5a19682d..6ca59e8e 100644
--- a/src/menu/control.cpp
+++ b/src/menu/control.cpp
@@ -39,8 +39,8 @@ namespace usdx
 		// you need the next iterator to be able to produce the next value after
 		// removing it (each control, will be issue a request to remove it self
 		// from the list)
-		for (std::list<Control*>::iterator next = slaves.begin(), it = next++;
-			 it != slaves.end(); it = next++) {
+		for (std::list<const Control*>::iterator next = slaves.begin(),
+			 it = next++; it != slaves.end(); it = next++) {
 			delete *it;
 		}
 
@@ -65,17 +65,17 @@ namespace usdx
 		}
 	}
 
-	Control* Control::get_owner(void) const
+	const Control* Control::get_owner(void) const
 	{
 		return owner;
 	}
 
-	void Control::add_slave(Control *new_slave)
+	void Control::add_slave(const Control *new_slave)
 	{
 		slaves.push_back(new_slave);
 	}
 
-	void Control::remove_slave(Control *slave)
+	void Control::remove_slave(const Control *slave)
 	{
 		slaves.remove(slave);
 	}
diff --git a/src/menu/control.hpp b/src/menu/control.hpp
index b821e79b..a4314579 100644
--- a/src/menu/control.hpp
+++ b/src/menu/control.hpp
@@ -43,10 +43,14 @@ namespace usdx
 		Control* owner;
 
 	protected:
-		std::list<Control*> slaves;
+		/**
+		 * All slaves the control owns. All these controls should be deleted
+		 * during decontruction of this control.
+		 */
+		std::list<const Control*> slaves;
 
-		void add_slave(Control*);
-		void remove_slave(Control*);
+		void add_slave(const Control*);
+		void remove_slave(const Control*);
 
 	public:
 		Control(Control*);
@@ -58,7 +62,11 @@ namespace usdx
 		 * withthe new owner.
 		 */
 		void set_owner(Control*);
-		Control* get_owner(void) const;
+
+		/**
+		 * Returns the current owner of this control.
+		 */
+		const Control* get_owner(void) const;
 	};
 };
 
diff --git a/src/utils/image.cpp b/src/utils/image.cpp
index 4146e0e4..93808f2e 100644
--- a/src/utils/image.cpp
+++ b/src/utils/image.cpp
@@ -58,7 +58,7 @@ namespace usdx
 		}
 	}
 
-	void Image::load(void)
+	void Image::load(void) const
 	{
 		BinaryFile file(filename);
 
@@ -73,7 +73,7 @@ namespace usdx
 		}
 	}
 
-	SDL_Surface* Image::get_surface(void)
+	const SDL_Surface* Image::get_surface(void) const
 	{
 		if (surface == NULL) {
 			load();
diff --git a/src/utils/image.hpp b/src/utils/image.hpp
index e27dc49d..b968652c 100644
--- a/src/utils/image.hpp
+++ b/src/utils/image.hpp
@@ -51,15 +51,15 @@ namespace usdx
 		static log4cpp::Category&  log;
 
 		boost::filesystem::wpath filename;
-		SDL_Surface* surface;
+		mutable SDL_Surface* surface;
 
-		void load(void);
+		void load(void) const;
 
 	public:
 		Image(boost::filesystem::wpath filename);
 		virtual ~Image();
 
-		SDL_Surface* get_surface(void);
+		const SDL_Surface* get_surface(void) const;
 	};
 };
 
-- 
cgit v1.2.3