Add LOVE_UNUSED macro to stamp out all those "unused parameter" warnings

src/common/Exception.cpp

@@ -19,6 +19,7 @@
 **/
 
 #include "Exception.h"
+#include <common/config.h>
 
 namespace love
 {
@@ -32,6 +33,7 @@ namespace love
 
 	Exception::Exception(int unparsed, const char * str)
 	{
+		LOVE_UNUSED(unparsed);
 		strncpy(buffer, str, BUFFER_SIZE);
 	}
 

src/common/config.h

@@ -48,6 +48,10 @@
 #	define _CRT_SECURE_NO_WARNINGS
 #endif
 
+#ifndef LOVE_UNUSED
+#	define LOVE_UNUSED(x) (void)sizeof(x)
+#endif
+
 #ifndef LOVE_BUILD
 #	define LOVE_BUILD
 #	define LOVE_BUILD_STANDALONE

src/modules/graphics/opengl/Graphics.cpp

@@ -734,6 +734,7 @@ namespace opengl
 		//	glEnable (GL_POLYGON_SMOOTH);
 		//	glHint (GL_POLYGON_SMOOTH_HINT, GL_NICEST);
 		//}
+		LOVE_UNUSED(style);
 	}
 
 	void Graphics::setLine( float width, Graphics::LineStyle style )

src/modules/graphics/opengl/VertexBuffer.cpp

@@ -21,6 +21,7 @@
 #include "VertexBuffer.h"
 
 #include "common/Exception.h"
+#include <common/config.h>
 
 #include <cstdlib>
 #include <cstring>
@@ -61,13 +62,13 @@ namespace opengl
 	}
 
 	// VertexBuffer::Bind
-
+
-	VertexBuffer::Bind::Bind(VertexBuffer &buf)
+	VertexBuffer::Bind::Bind(VertexBuffer &buf)
-		: buf(buf)
+		: buf(buf)
-	{
+	{
-		buf.bind();
+		buf.bind();
-	}
+	}
-
+
 	VertexBuffer::Bind::~Bind()
 	{
 		buf.unbind();
@@ -86,23 +87,24 @@ namespace opengl
 		delete [] buf;
 	}
 
-	void *VertexArray::map(GLenum access)
+	void *VertexArray::map(GLenum access)
-	{
+	{
-		return buf;
+		LOVE_UNUSED(access);
-	}
+		return buf;
-
+	}
-	void VertexArray::unmap()
+
-	{
+	void VertexArray::unmap()
-	}
+	{
-
+	}
-	void VertexArray::bind()
+
-	{
+	void VertexArray::bind()
-	}
+	{
-
+	}
-	void VertexArray::unbind()
+
-	{
+	void VertexArray::unbind()
-	}
+	{
-
+	}
+
 	void VertexArray::fill(int offset, int size, const void *data)
 	{
 		memcpy(buf + offset, data, size);
@@ -136,33 +138,33 @@ namespace opengl
 			unload(false);
 	}
 
-	void *VBO::map(GLenum access)
+	void *VBO::map(GLenum access)
-	{
+	{
-		// Don't map twice.
+		// Don't map twice.
-		if (mapped)
+		if (mapped)
-			return mapped;
+			return mapped;
-
+
-		mapped = glMapBufferARB(getTarget(), access);
+		mapped = glMapBufferARB(getTarget(), access);
-
+
-		return mapped;
+		return mapped;
-	}
+	}
-
+
-	void VBO::unmap()
+	void VBO::unmap()
-	{
+	{
-		glUnmapBufferARB(getTarget());
+		glUnmapBufferARB(getTarget());
-		mapped = 0;
+		mapped = 0;
-	}
+	}
-
+
-	void VBO::bind()
+	void VBO::bind()
-	{
+	{
-		glBindBufferARB(getTarget(), vbo);
+		glBindBufferARB(getTarget(), vbo);
-	}
+	}
-
+
-	void VBO::unbind()
+	void VBO::unbind()
-	{
+	{
-		glBindBufferARB(getTarget(), 0);
+		glBindBufferARB(getTarget(), 0);
-	}
+	}
-
+
 	void VBO::fill(int offset, int size, const void *data)
 	{
 		if (mapped)
@@ -176,35 +178,35 @@ namespace opengl
 		return reinterpret_cast<const void*>(offset);
 	}
 
-	bool VBO::loadVolatile()
+	bool VBO::loadVolatile()
-	{
+	{
-		return load(true);
+		return load(true);
-	}
-
-	void VBO::unloadVolatile()
-	{
-		unload(true);
 	}
 
-	bool VBO::load(bool restore)
+	void VBO::unloadVolatile()
-	{
+	{
-		glGenBuffersARB(1, &vbo);
+		unload(true);
-
+	}
-		VertexBuffer::Bind bind(*this);
+
-
+	bool VBO::load(bool restore)
-		// Copy the old buffer only if 'restore' was requested.
+	{
-		const GLvoid *src = restore ? buffer_copy : 0;
+		glGenBuffersARB(1, &vbo);
-		
+
-		// Note that if 'src' is '0', no data will be copied.
+		VertexBuffer::Bind bind(*this);
-		glBufferDataARB(getTarget(), getSize(), src, getUsage());
+
-
+		// Copy the old buffer only if 'restore' was requested.
+		const GLvoid *src = restore ? buffer_copy : 0;
+		
+		// Note that if 'src' is '0', no data will be copied.
+		glBufferDataARB(getTarget(), getSize(), src, getUsage());
+
 		// Clean up buffer_copy, if it exists.
 		delete[] buffer_copy;
-		buffer_copy = 0;
+		buffer_copy = 0;
-
+
-		return true;
+		return true;
-	}
+	}
-
+
 	void VBO::unload(bool save)
 	{
 		// Clean up buffer_copy, if it exists.

src/modules/thread/sdl/threads.h

@@ -22,6 +22,7 @@
 #define LOVE_THREAD_SDL_THREADS_H
 
 #include "SDL.h"
+#include <common/config.h>
 
 namespace love
 {
@@ -31,7 +32,7 @@ namespace thread
 	class Mutex {
 	private:
 		SDL_mutex* mutex;
-		Mutex(const Mutex& mutex) {}
+		Mutex(const Mutex& mutex) {LOVE_UNUSED(mutex);}
 
 		friend class Conditional;
 
@@ -48,7 +49,7 @@ namespace thread
 	class ThreadBase {
 	private:
 		SDL_Thread* thread;
-		ThreadBase(ThreadBase& thread) {}
+		ThreadBase(ThreadBase& thread) {LOVE_UNUSED(thread);}
 		bool running;
 
 		static int thread_runner(void* param);
@@ -70,7 +71,7 @@ namespace thread
 
 	class Semaphore {
 	private:
-		Semaphore(const Semaphore& sem) {}
+		Semaphore(const Semaphore& sem) {LOVE_UNUSED(sem);}
 		SDL_sem* semaphore;
 
 	public:

src/modules/thread/threads.h

@@ -63,7 +63,7 @@ namespace thread
 	private:
 		Mutex* mutex;
 
-		Lock(Lock& lock) {}
+		Lock(Lock& lock) {LOVE_UNUSED(lock);}
 
 	public:
 		Lock(Mutex* m): mutex(m) {