Merge branch 'develop', Major API expansion

.gitignore

@@ -15,7 +15,6 @@ Out/
 *.o
 *.a
 *.exe
-*.diff
 
 #Shell files
 *.bat

client/character.hpp

@@ -1,86 +0,0 @@
-/* Copyright: (c) Kayne Ruse 2014
- * 
- * This software is provided 'as-is', without any express or implied
- * warranty. In no event will the authors be held liable for any damages
- * arising from the use of this software.
- * 
- * Permission is granted to anyone to use this software for any purpose,
- * including commercial applications, and to alter it and redistribute it
- * freely, subject to the following restrictions:
- * 
- * 1. The origin of this software must not be misrepresented; you must not
- * claim that you wrote the original software. If you use this software
- * in a product, an acknowledgment in the product documentation would be
- * appreciated but is not required.
- * 
- * 2. Altered source versions must be plainly marked as such, and must not be
- * misrepresented as being the original software.
- * 
- * 3. This notice may not be removed or altered from any source
- * distribution.
-*/
-#ifndef CHARACTER_HPP_
-#define CHARACTER_HPP_
-
-//components
-#include "character_defines.hpp"
-#include "vector2.hpp"
-#include "bounding_box.hpp"
-
-//graphics
-#include "sprite_sheet.hpp"
-
-//std namespace
-#include <string>
-#include <cmath>
-
-class Character {
-public:
-	Character() = default;
-	~Character() = default;
-
-	void Update(double delta);
-
-	//graphics
-	void DrawTo(SDL_Surface* const, int camX, int camY);
-	void CorrectSprite();
-	SpriteSheet* GetSprite() { return &sprite; }
-
-	//accessors and mutators
-
-	//metadata
-	int SetOwner(int i) { return owner = i; }
-	int GetOwner() { return owner; }
-	std::string SetHandle(std::string s) { return handle = s; }
-	std::string GetHandle() const { return handle; }
-	std::string SetAvatar(std::string s) { return avatar = s; }
-	std::string GetAvatar() const { return avatar; }
-
-	//position
-	Vector2 SetOrigin(Vector2 v) { return origin = v; }
-	Vector2 GetOrigin() const { return origin; }
-	Vector2 SetMotion(Vector2 v) { return motion = v; }
-	Vector2 GetMotion() const { return motion; }
-	BoundingBox SetBoundingBox(BoundingBox b) { return bounds = b; }
-	BoundingBox GetBoundingBox() const { return bounds; }
-
-private:
-	//graphics
-	SpriteSheet sprite;
-
-	//metadata
-	int owner;
-	std::string handle;
-	std::string avatar;
-
-	//position
-	Vector2 origin = {0.0,0.0};
-	Vector2 motion = {0.0,0.0};
-	BoundingBox bounds;
-};
-
-//tmp
-#include <map>
-typedef std::map<int, Character> CharacterMap;
-
-#endif

client/clean_up.cpp

@@ -29,43 +29,29 @@
 //Public access members
 //-------------------------
 
-CleanUp::CleanUp(lua_State* L, UDPNetworkUtility& aNetwork, CharacterMap& aCharacterMap):
-	lua(L),
-	network(aNetwork),
-	characterMap(aCharacterMap)
+CleanUp::CleanUp(
+	ConfigUtility* const argConfig,
+	UDPNetworkUtility* const argNetwork,
+	int* const argClientIndex,
+	int* const argAccountIndex,
+	int* const argCharacterIndex,
+	std::map<int, CombatData>* argCombatMap,
+	std::map<int, CharacterData>* argCharacterMap,
+	std::map<int, EnemyData>* argEnemyMap
+	):
+	config(*argConfig),
+	network(*argNetwork),
+	clientIndex(*argClientIndex),
+	accountIndex(*argAccountIndex),
+	characterIndex(*argCharacterIndex),
+	combatMap(*argCombatMap),
+	characterMap(*argCharacterMap),
+	enemyMap(*argEnemyMap)
 {
-	//get the config table
-	lua_getglobal(lua, "config");
-
-	//TODO: I need to figure out an alternative to loading these over and over again
-	//get the directories
-	lua_getfield(lua, -1, "dir");
-	lua_getfield(lua, -1, "interface");
-	lua_getfield(lua, -2, "fonts");
-
-	std::string interfaceDir = lua_tostring(lua, -2);
-	std::string fontsDir = lua_tostring(lua, -1);
-
-	lua_pop(lua, 3);
-
-	//clear the indicies
-	lua_getfield(lua, -1, "client");
-
-	lua_pushnil(lua);
-	lua_pushnil(lua);
-	lua_pushnil(lua);
-
-	lua_setfield(lua, -4, "clientIndex");
-	lua_setfield(lua, -3, "accountIndex");
-	lua_setfield(lua, -2, "characterIndex");
-
-	//pop the remaining objects from the stack
-	lua_pop(lua, 2);
-
 	//setup the utility objects
-	image.LoadSurface(interfaceDir + "button_menu.bmp");
+	image.LoadSurface(config["dir.interface"] + "button_menu.bmp");
 	image.SetClipH(image.GetClipH()/3);
-	font.LoadSurface(fontsDir + "pk_white_8.bmp");
+	font.LoadSurface(config["dir.fonts"] + "pk_white_8.bmp");
 
 	//pass the utility objects
 	backButton.SetImage(&image);
@@ -80,7 +66,12 @@ CleanUp::CleanUp(lua_State* L, UDPNetworkUtility& aNetwork, CharacterMap& aChara
 
 	//full reset
 	network.Unbind(Channels::SERVER);
+	clientIndex = -1;
+	accountIndex = -1;
+	characterIndex = -1;
+	combatMap.clear();
 	characterMap.clear();
+	enemyMap.clear();
 
 	//auto return
 	startTick = std::chrono::steady_clock::now();
@@ -96,7 +87,7 @@ CleanUp::~CleanUp() {
 
 void CleanUp::Update(double delta) {
 	if (std::chrono::steady_clock::now() - startTick > std::chrono::duration<int>(10)) {
-		SetNextScene(SceneList::MAINMENU);
+		QuitEvent();
 	}
 
 	//BUGFIX: Eat incoming packets
@@ -132,8 +123,7 @@ void CleanUp::MouseButtonDown(SDL_MouseButtonEvent const& button) {
 }
 
 void CleanUp::MouseButtonUp(SDL_MouseButtonEvent const& button) {
-	if (backButton.MouseButtonUp(button) == Button::State::HOVER &&
-		button.button & SDL_BUTTON_LMASK) {
+	if (backButton.MouseButtonUp(button) == Button::State::HOVER) {
 		SetNextScene(SceneList::MAINMENU);
 	}
 }

client/clean_up.hpp

@@ -25,18 +25,21 @@
 //network
 #include "udp_network_utility.hpp"
 
-//graphics & ui
+//graphics
 #include "image.hpp"
 #include "raster_font.hpp"
 #include "button.hpp"
+
+//common
+#include "config_utility.hpp"
 #include "frame_rate.hpp"
 
+#include "combat_data.hpp"
+#include "character_data.hpp"
+#include "enemy_data.hpp"
+
 //client
 #include "base_scene.hpp"
-#include "character.hpp"
-
-//APIs
-#include "lua/lua.hpp"
 
 //std namespace
 #include <chrono>
@@ -44,7 +47,16 @@
 class CleanUp : public BaseScene {
 public:
 	//Public access members
-	CleanUp(lua_State*, UDPNetworkUtility&, CharacterMap&);
+	CleanUp(
+		ConfigUtility* const argConfig,
+		UDPNetworkUtility* const argNetwork,
+		int* const argClientIndex,
+		int* const argAccountIndex,
+		int* const argCharacterIndex,
+		std::map<int, CombatData>* argCombatMap,
+		std::map<int, CharacterData>* argCharacterMap,
+		std::map<int, EnemyData>* argEnemyMap
+	);
 	~CleanUp();
 
 protected:
@@ -62,13 +74,20 @@ protected:
 	void KeyUp(SDL_KeyboardEvent const&);
 
 	//shared parameters
-	lua_State* lua = nullptr;
+	ConfigUtility& config;
 	UDPNetworkUtility& network;
-	CharacterMap& characterMap;
+	int& clientIndex;
+	int& accountIndex;
+	int& characterIndex;
+	std::map<int, CombatData>& combatMap;
+	std::map<int, CharacterData>& characterMap;
+	std::map<int, EnemyData>& enemyMap;
 
-	//graphics & ui
+	//graphics
 	Image image;
 	RasterFont font;
+
+	//UI
 	Button backButton;
 	FrameRate fps;
 

client/client_application.cpp

@@ -25,7 +25,6 @@
 
 #include <stdexcept>
 #include <chrono>
-#include <iostream>
 
 //-------------------------
 //Scene headers
@@ -45,106 +44,24 @@
 //-------------------------
 
 void ClientApplication::Init(int argc, char** argv) {
-	std::cout << "Beginning " << argv[0] << std::endl;
-
-	//-------------------------
-	//Initialize the APIs
-	//-------------------------
+	//load the prerequisites
+	config.Load("rsc\\config.cfg");
 
 	//initialize SDL
 	if (SDL_Init(SDL_INIT_VIDEO)) {
 		throw(std::runtime_error("Failed to initialize SDL"));
 	}
-	std::cout << "Initialized SDL" << std::endl;
+	int w = config.Int("client.screen.w");
+	int h = config.Int("client.screen.h");
+	int f = config.Bool("client.screen.f") ? SDL_HWSURFACE|SDL_DOUBLEBUF|SDL_FULLSCREEN : SDL_HWSURFACE|SDL_DOUBLEBUF;
+
+	BaseScene::SetScreen(w ? w : 800, h ? h : 600, 0, f);
 
 	//initialize SDL_net
 	if (SDLNet_Init()) {
 		throw(std::runtime_error("Failed to initialize SDL_net"));
 	}
 	network.Open(0);
-	std::cout << "Initialized SDL_net" << std::endl;
-
-	//initialize lua
-	lua = luaL_newstate();
-	if (!lua) {
-		throw(std::runtime_error("Failed to initialize lua"));
-	}
-	luaL_openlibs(lua);
-	std::cout << "Initialized lua" << std::endl;
-
-	//run the setup script
-	if (luaL_dofile(lua, "rsc\\setup.lua")) {
-		throw(std::runtime_error("Failed to initialize lua's startup script"));
-	}
-	std::cout << "Initialized lua's setup script" << std::endl;
-
-	//place the config table onto the stack
-	lua_getglobal(lua, "config");
-
-	//-------------------------
-	//Setup the screen
-	//-------------------------
-
-	//get each field
-	lua_getfield(lua, -1, "client");
-	lua_getfield(lua, -1, "screen");
-	lua_getfield(lua, -1, "width");
-	lua_getfield(lua, -2, "height");
-	lua_getfield(lua, -3, "fullscreen");
-
-	int w = lua_tointeger(lua, -3);
-	int h = lua_tointeger(lua, -2);
-	int f = lua_toboolean(lua, -1);
-
-	//pop the screen members
-	lua_pop(lua, 5);
-
-	BaseScene::SetScreen(w ? w : 800, h ? h : 600, 0, f ?
-		SDL_HWSURFACE|SDL_DOUBLEBUF|SDL_FULLSCREEN :
-		SDL_HWSURFACE|SDL_DOUBLEBUF);
-	std::cout << "Initialized the screen" << std::endl;
-
-	//-------------------------
-	//debug output
-	//-------------------------
-
-	lua_getfield(lua, -1, "debug");
-
-	if (lua_toboolean(lua, -1)) {
-#define DEBUG_OUTPUT_VAR(x) std::cout << "\t" << #x << ": " << x << std::endl;
-
-		std::cout << "Internal sizes:" << std::endl;
-
-		DEBUG_OUTPUT_VAR(sizeof(Region::type_t));
-		DEBUG_OUTPUT_VAR(sizeof(Region));
-		DEBUG_OUTPUT_VAR(REGION_WIDTH);
-		DEBUG_OUTPUT_VAR(REGION_HEIGHT);
-		DEBUG_OUTPUT_VAR(REGION_DEPTH);
-		DEBUG_OUTPUT_VAR(REGION_SOLID_FOOTPRINT);
-		DEBUG_OUTPUT_VAR(REGION_FOOTPRINT);
-		DEBUG_OUTPUT_VAR(PACKET_BUFFER_SIZE);
-		DEBUG_OUTPUT_VAR(MAX_PACKET_SIZE);
-
-#undef DEBUG_OUTPUT_VAR
-	}
-
-	//pop the debug value
-	lua_pop(lua, 1);
-
-	//-------------------------
-	//finalize the startup
-	//-------------------------
-
-	//pop the config table
-	lua_pop(lua, 1);
-
-	std::cout << "Startup completed successfully" << std::endl;
-
-	//-------------------------
-	//debugging
-	//-------------------------
-
-	//...
 }
 
 void ClientApplication::Proc() {
@@ -171,7 +88,7 @@ void ClientApplication::Proc() {
 		//simulate game time
 		while (simTime < realTime) {
 			//call each user defined function
-			activeScene->RunFrame(constexpr(double(delta.count()) / std::chrono::duration<int, std::milli>::period::den));
+			activeScene->RunFrame(double(delta.count()) / std::chrono::duration<int, std::milli>::period::den);
 			simTime += delta;
 		}
 
@@ -183,12 +100,9 @@ void ClientApplication::Proc() {
 }
 
 void ClientApplication::Quit() {
-	std::cout << "Shutting down" << std::endl;
-	lua_close(lua);
 	network.Close();
 	SDLNet_Quit();
 	SDL_Quit();
-	std::cout << "Clean exit" << std::endl;
 }
 
 //-------------------------
@@ -201,25 +115,25 @@ void ClientApplication::LoadScene(SceneList sceneIndex) {
 		//add scene creation calls here
 		case SceneList::FIRST:
 		case SceneList::SPLASHSCREEN:
-			activeScene = new SplashScreen(lua);
+			activeScene = new SplashScreen(&config);
 		break;
 		case SceneList::MAINMENU:
-			activeScene = new MainMenu(lua);
+			activeScene = new MainMenu(&config);
 		break;
 		case SceneList::OPTIONSMENU:
-			activeScene = new OptionsMenu(lua);
+			activeScene = new OptionsMenu(&config);
 		break;
 		case SceneList::LOBBYMENU:
-			activeScene = new LobbyMenu(lua, network, characterMap);
+			activeScene = new LobbyMenu(&config, &network, &clientIndex, &accountIndex);
 		break;
 		case SceneList::INWORLD:
-			activeScene = new InWorld(lua, network, characterMap);
+			activeScene = new InWorld(&config, &network, &clientIndex, &accountIndex, &characterIndex, &combatMap, &characterMap);
 		break;
 		case SceneList::INCOMBAT:
-			activeScene = new InCombat(lua, network, characterMap);
+			activeScene = new InCombat(&config, &network, &clientIndex, &accountIndex, &characterIndex, &combatMap, &characterMap, &enemyMap);
 		break;
 		case SceneList::CLEANUP:
-			activeScene = new CleanUp(lua, network, characterMap);
+			activeScene = new CleanUp(&config, &network, &clientIndex, &accountIndex, &characterIndex, &combatMap, &characterMap, &enemyMap);
 		break;
 		default:
 			throw(std::logic_error("Failed to recognize the scene index"));

client/client_application.hpp

@@ -25,10 +25,11 @@
 #include "scene_list.hpp"
 #include "base_scene.hpp"
 
+#include "config_utility.hpp"
 #include "udp_network_utility.hpp"
-#include "character.hpp"
-
-#include "lua/lua.hpp"
+#include "character_data.hpp"
+#include "combat_data.hpp"
+#include "enemy_data.hpp"
 
 #include <map>
 
@@ -49,13 +50,15 @@ private:
 	BaseScene* activeScene = nullptr;
 
 	//shared parameters
-	lua_State* lua = nullptr;
+	ConfigUtility config;
 	UDPNetworkUtility network;
 	int clientIndex = -1;
 	int accountIndex = -1;
 	int characterIndex = -1;
 
-	CharacterMap characterMap;
+	std::map<int, CombatData> combatMap;
+	std::map<int, CharacterData> characterMap;
+	std::map<int, EnemyData> enemyMap;
 };
 
 #endif

client/in_combat.cpp

@@ -36,14 +36,18 @@ InCombat::InCombat(
 	int* const argClientIndex,
 	int* const argAccountIndex,
 	int* const argCharacterIndex,
-	CharacterMap* argCharacterMap
+	std::map<int, CombatData>* argCombatMap,
+	std::map<int, CharacterData>* argCharacterMap,
+	std::map<int, EnemyData>* argEnemyMap
 	):
 	config(*argConfig),
 	network(*argNetwork),
 	clientIndex(*argClientIndex),
 	accountIndex(*argAccountIndex),
 	characterIndex(*argCharacterIndex),
-	characterMap(*argCharacterMap)
+	combatMap(*argCombatMap),
+	characterMap(*argCharacterMap),
+	enemyMap(*argEnemyMap)
 {
 /*	//setup the utility objects
 	buttonImage.LoadSurface(config["dir.interface"] + "button_menu.bmp");

client/in_combat.hpp

@@ -34,7 +34,9 @@
 #include "config_utility.hpp"
 #include "frame_rate.hpp"
 
-#include "character.hpp"
+#include "combat_data.hpp"
+#include "character_data.hpp"
+#include "enemy_data.hpp"
 
 //client
 #include "base_scene.hpp"
@@ -48,7 +50,9 @@ public:
 		int* const argClientIndex,
 		int* const argAccountIndex,
 		int* const argCharacterIndex,
-		CharacterMap* argCharacterMap
+		std::map<int, CombatData>* argCombatMap,
+		std::map<int, CharacterData>* argCharacterMap,
+		std::map<int, EnemyData>* argEnemyMap
 	);
 	~InCombat();
 
@@ -71,12 +75,14 @@ protected:
 	//Network handlers
 	void HandlePacket(SerialPacket* const);
 	void HandleDisconnect(SerialPacket* const);
+	//TODO: more network handlers
 
 	//Server control
 	void RequestSynchronize();
 	void SendPlayerUpdate();
 	void RequestDisconnect();
 	void RequestShutdown();
+	//TODO: more
 
 	//shared parameters
 	ConfigUtility& config;
@@ -84,7 +90,9 @@ protected:
 	int& clientIndex;
 	int& accountIndex;
 	int& characterIndex;
-	CharacterMap& characterMap;
+	std::map<int, CombatData>& combatMap;
+	std::map<int, CharacterData>& characterMap;
+	std::map<int, EnemyData>& enemyMap;
 
 	//graphics
 	//TODO: graphics

client/in_world.cpp

@@ -24,10 +24,9 @@
 #include "channels.hpp"
 #include "utility.hpp"
 
-#include <stdexcept>
 #include <algorithm>
 #include <cmath>
-#include <iostream>
+#include <stdexcept>
 
 //-------------------------
 //Public access members
@@ -39,69 +38,17 @@ InWorld::InWorld(
 	int* const argClientIndex,
 	int* const argAccountIndex,
 	int* const argCharacterIndex,
-	CharacterMap* argCharacterMap
+	std::map<int, CombatData>* argCombatMap,
+	std::map<int, CharacterData>* argCharacterMap
 	):
 	config(*argConfig),
 	network(*argNetwork),
 	clientIndex(*argClientIndex),
 	accountIndex(*argAccountIndex),
 	characterIndex(*argCharacterIndex),
+	combatMap(*argCombatMap),
 	characterMap(*argCharacterMap)
 {
-	//register the pager
-	lua_pushstring(lua, TORTUGA_REGION_PAGER_PSEUDO_INDEX);
-	lua_pushlightuserdata(lua, &pager);
-	lua_settable(L, LUA_REGISTRYINDEX);
-
-	//register the tilesheet
-	lua_pushstring(lua, TORTUGA_TILE_SHEET_PSEUDO_INDEX);
-	lua_pushlightuserdata(lua, &tileSheet);
-	lua_settable(L, LUA_REGISTRYINDEX);
-
-	//setup the component objecrs
-	pager.SetLuaState(lua);
-
-	//get the config info
-	lua_getglobal(lua, "config");
-	lua_getfield(lua, -1, "dir");
-	lua_getfield(lua, -1, "fonts");
-	std::string fonts = lua_tostring(lua, -1);
-	lua_getfield(lua, -2, "interface");
-	std::string interface = lua_tostring(lua, -1);
-	lua_getfield(lua, -3, "sprites");
-	std::string sprites = lua_tostring(lua, -1);
-	lua_getfield(lua, -4, "scripts");
-	std::string scripts = lua_tostring(lua, -1);
-	lua_pop(lua, 6);
-
-	//run the additional scripts
-	if (luaL_dofile(lua, (scripts + "in_world.lua").c_str())) {
-		throw(std::runtime_error(std::string() + "Failed to run in_world.lua: " + lua_tostring(lua, -1) ));
-	}
-
-	//setup the utility objects
-	buttonImage.LoadSurface(interface + "button_menu.bmp");
-	buttonImage.SetClipH(buttonImage.GetClipH()/3);
-	font.LoadSurface(fonts + "pk_white_8.bmp");
-
-	//pass the utility objects
-	backButton.SetImage(&buttonImage);
-	backButton.SetFont(&font);
-
-	//set the button positions
-	backButton.SetX(50);
-	backButton.SetY(50 + buttonImage.GetClipH() * 0);
-
-	//set the button texts
-	backButton.SetText("Back");
-
-	//entities
-	character.GetSprite()->LoadSurface(sprites + "elliot2.bmp", 4, 4);
-	character.SetBoundingBox({0, 0,
-		character.GetSprite()->GetImage()->GetClipW(),
-		character.GetSprite()->GetImage()->GetClipH()
-	});
-//-------------------------
 	//setup the utility objects
 	buttonImage.LoadSurface(config["dir.interface"] + "button_menu.bmp");
 	buttonImage.SetClipH(buttonImage.GetClipH()/3);
@@ -135,13 +82,7 @@ InWorld::InWorld(
 }
 
 InWorld::~InWorld() {
-	//unregister the map components
-	lua_pushstring(lua, TORTUGA_REGION_PAGER_PSEUDO_INDEX);
-	lua_pushstring(lua, TORTUGA_TILE_SHEET_PSEUDO_INDEX);
-	lua_pushnil(lua);
-	lua_settable(lua, LUA_REGISTRYINDEX);
-	lua_pushnil(lua);
-	lua_settable(lua, LUA_REGISTRYINDEX);
+	//
 }
 
 //-------------------------
@@ -165,34 +106,10 @@ void InWorld::Update(double delta) {
 		it.second.Update(delta);
 	}
 
-	//TODO: Check collisions here
-	//check for collisions with the map
-	BoundingBox wallBounds = {0, 0, tileSheet.GetTileW(), tileSheet.GetTileH()};
-	const int xCount = character.GetBoundingBox().w / wallBounds.w + 1;
-	const int yCount = character.GetBoundingBox().h / wallBounds.h + 1;
-
-	for (int i = -1; i <= xCount; ++i) {
-		for (int j = -1; j <= yCount; ++j) {
-			//set the wall's position
-			wallBounds.x = wallBounds.w * i + snapToBase((double)wallBounds.w, character.GetOrigin().x);
-			wallBounds.y = wallBounds.h * j + snapToBase((double)wallBounds.h, character.GetOrigin().y);
-
-			if (!pager.GetSolid(wallBounds.x / wallBounds.w, wallBounds.y / wallBounds.h)) {
-				continue;
-			}
-
-			if ((character.GetOrigin() + character.GetBoundingBox()).CheckOverlap(wallBounds)) {
-				character.SetOrigin(character.GetOrigin() - (character.GetMotion() * delta));
-				character.SetMotion({0,0});
-				character.CorrectSprite();
-			}
-		}
-	}
-
 	//update the camera
 	if(localCharacter) {
-		camera.x = localCharacter->GetOrigin().x - camera.marginX;
-		camera.y = localCharacter->GetOrigin().y - camera.marginY;
+		camera.x = localCharacter->origin.x - camera.marginX;
+		camera.y = localCharacter->origin.y - camera.marginY;
 	}
 
 	//check the map
@@ -258,60 +175,77 @@ void InWorld::MouseButtonUp(SDL_MouseButtonEvent const& button) {
 }
 
 void InWorld::KeyDown(SDL_KeyboardEvent const& key) {
-	if (!localCharacter) {
-		return;
-	}
-
-	//player movement
-	Vector2 motion = localCharacter->GetMotion();
 	switch(key.keysym.sym) {
+		//player movement
 		case SDLK_LEFT:
-			motion.x -= CHARACTER_WALKING_SPEED;
+			if (localCharacter) {
+				localCharacter->motion.x -= CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
+
 		case SDLK_RIGHT:
-			motion.x += CHARACTER_WALKING_SPEED;
+			if (localCharacter) {
+				localCharacter->motion.x += CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
+
 		case SDLK_UP:
-			motion.y -= CHARACTER_WALKING_SPEED;
+			if (localCharacter) {
+				localCharacter->motion.y -= CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
+
 		case SDLK_DOWN:
-			motion.y += CHARACTER_WALKING_SPEED;
+			if (localCharacter) {
+				localCharacter->motion.y += CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
-		default:
-			return;
 	}
-	localCharacter->SetMotion(motion);
-	localCharacter->CorrectSprite();
-	SendPlayerUpdate();
 }
 
 void InWorld::KeyUp(SDL_KeyboardEvent const& key) {
-	if (!localCharacter) {
-		return;
-	}
-
-	//player movement
-	Vector2 motion = localCharacter->GetMotion();
 	switch(key.keysym.sym) {
-		//NOTE: The use of min/max here are to prevent awkward movements
+		//player movement
 		case SDLK_LEFT:
-			motion.x = std::min(motion.x + CHARACTER_WALKING_SPEED, 0.0);
+			if (localCharacter) {
+				localCharacter->motion.x += CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
+
 		case SDLK_RIGHT:
-			motion.x = std::max(motion.x - CHARACTER_WALKING_SPEED, 0.0);
+			if (localCharacter) {
+				localCharacter->motion.x -= CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
+
 		case SDLK_UP:
-			motion.y = std::min(motion.y + CHARACTER_WALKING_SPEED, 0.0);
+			if (localCharacter) {
+				localCharacter->motion.y += CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
+
 		case SDLK_DOWN:
-			motion.y = std::max(motion.y - CHARACTER_WALKING_SPEED, 0.0);
+			if (localCharacter) {
+				localCharacter->motion.y -= CHARACTER_WALKING_SPEED;
+				localCharacter->CorrectSprite();
+				SendPlayerUpdate();
+			}
 		break;
-		default:
-			return;
 	}
-	localCharacter->SetMotion(motion);
-	localCharacter->CorrectSprite();
-	SendPlayerUpdate();
 }
 
 //-------------------------
@@ -352,41 +286,36 @@ void InWorld::HandleCharacterNew(CharacterPacket* const argPacket) {
 	}
 
 	//create the character object
-	Character& newCharacter = characterMap[argPacket->characterIndex];
+	CharacterData& character = characterMap[argPacket->characterIndex];
 
-	//fill out the character's members
-	newCharacter.SetHandle(argPacket->handle);
-	newCharacter.SetAvatar(argPacket->avatar);
+	//set the members
+	character.handle = argPacket->handle;
+	character.avatar = argPacket->avatar;
+	character.sprite.LoadSurface(config["dir.sprites"] + character.avatar, 4, 4);
+	character.roomIndex = argPacket->roomIndex;
+	character.origin = argPacket->origin;
+	character.motion = argPacket->motion;
+	character.stats = argPacket->stats;
 
-	newCharacter.GetSprite()->LoadSurface(config["dir.sprites"] + newCharacter.GetAvatar(), 4, 4);
-
-	newCharacter.SetOrigin(argPacket->origin);
-	newCharacter.SetMotion(argPacket->motion);
-
-	(*newCharacter.GetStats()) = argPacket->stats;
-
-	//bookkeeping code
-	newCharacter.CorrectSprite();
+	character.CorrectSprite();
 
 	//catch this client's player object
 	if (argPacket->accountIndex == accountIndex && !localCharacter) {
 		characterIndex = argPacket->characterIndex;
-		localCharacter = &newCharacter;
+		localCharacter = &character;
 
 		//setup the camera
-		//TODO: move this?
 		camera.width = GetScreen()->w;
 		camera.height = GetScreen()->h;
 
 		//center on the player's character
-		camera.marginX = (GetScreen()->w / 2 - localCharacter->GetSprite()->GetImage()->GetClipW() / 2);
-		camera.marginY = (GetScreen()->h / 2 - localCharacter->GetSprite()->GetImage()->GetClipH() / 2);
+		camera.marginX = (GetScreen()->w / 2 - localCharacter->sprite.GetImage()->GetClipW() / 2);
+		camera.marginY = (GetScreen()->h / 2 - localCharacter->sprite.GetImage()->GetClipH() / 2);
 	}
 }
 
 void InWorld::HandleCharacterDelete(CharacterPacket* const argPacket) {
 	//TODO: authenticate when own character is being deleted (linked to a TODO in the server)
-
 	//catch this client's player object
 	if (argPacket->characterIndex == characterIndex) {
 		characterIndex = -1;
@@ -398,17 +327,17 @@ void InWorld::HandleCharacterDelete(CharacterPacket* const argPacket) {
 
 void InWorld::HandleCharacterUpdate(CharacterPacket* const argPacket) {
 	if (characterMap.find(argPacket->characterIndex) == characterMap.end()) {
-		std::cout << "Warning: HandleCharacterUpdate() is passing to HandleCharacterNew()" << std::endl;
 		HandleCharacterNew(argPacket);
 		return;
 	}
 
-	Character& character = characterMap[argPacket->characterIndex];
+	CharacterData& character = characterMap[argPacket->characterIndex];
 
 	//other characters moving
 	if (argPacket->characterIndex != characterIndex) {
-		character.SetOrigin(argPacket->origin);
-		character.SetMotion(argPacket->motion);
+		character.roomIndex = argPacket->roomIndex;
+		character.origin = argPacket->origin;
+		character.motion = argPacket->motion;
 		character.CorrectSprite();
 	}
 }
@@ -435,8 +364,6 @@ void InWorld::RequestSynchronize() {
 	newPacket.clientIndex = clientIndex;
 	newPacket.accountIndex = accountIndex;
 
-	//TODO: location, range for sync request
-
 	network.SendTo(Channels::SERVER, &newPacket);
 }
 
@@ -447,12 +374,12 @@ void InWorld::SendPlayerUpdate() {
 	newPacket.type = SerialPacketType::CHARACTER_UPDATE;
 
 	newPacket.characterIndex = characterIndex;
-	//NOTE: omitting the handle and avatar here
+	//handle, avatar
 	newPacket.accountIndex = accountIndex;
-	newPacket.roomIndex = 0; //TODO: room index
-	newPacket.origin = localCharacter->GetOrigin();
-	newPacket.motion = localCharacter->GetMotion();
-	newPacket.stats = *localCharacter->GetStats();
+	newPacket.roomIndex = localCharacter->roomIndex;
+	newPacket.origin = localCharacter->origin;
+	newPacket.motion = localCharacter->motion;
+	newPacket.stats = localCharacter->stats;
 
 	//TODO: gameplay components: equipment, items, buffs, debuffs
 
@@ -508,7 +435,7 @@ void InWorld::UpdateMap() {
 
 	//prune distant regions
 	for (std::list<Region>::iterator it = regionPager.GetContainer()->begin(); it != regionPager.GetContainer()->end(); /* EMPTY */) {
-		//check if the region is outside of this area
+		//check if the region is outside off this area
 		if (it->GetX() < xStart || it->GetX() > xEnd || it->GetY() < yStart || it->GetY() > yEnd) {
 
 			//clunky, but the alternative was time consuming

client/in_world.hpp

@@ -22,28 +22,27 @@
 #ifndef INWORLD_HPP_
 #define INWORLD_HPP_
 
-//map stuff
-#include "tile_sheet.hpp"
-#include "region_pager_lua.hpp"
+//maps
+#include "region_pager_base.hpp"
 
 //networking
 #include "udp_network_utility.hpp"
 
-//graphics & ui
+//graphics
 #include "image.hpp"
 #include "raster_font.hpp"
 #include "button.hpp"
+#include "tile_sheet.hpp"
 
-//utilities
+//common
+#include "config_utility.hpp"
 #include "frame_rate.hpp"
-#include "timer.hpp"
+
+#include "combat_data.hpp"
+#include "character_data.hpp"
 
 //client
 #include "base_scene.hpp"
-#include "character.hpp"
-
-//APIs
-#include "lua/lua.hpp"
 
 //STL
 #include <map>
@@ -57,7 +56,8 @@ public:
 		int* const argClientIndex,
 		int* const argAccountIndex,
 		int* const argCharacterIndex,
-		CharacterMap* argCharacterMap
+		std::map<int, CombatData>* argCombatMap,
+		std::map<int, CharacterData>* argCharacterMap
 	);
 	~InWorld();
 
@@ -95,14 +95,14 @@ protected:
 	//utilities
 	void UpdateMap();
 
-	//TODO: Streamline this with lua
 	//shared parameters
 	ConfigUtility& config;
 	UDPNetworkUtility& network;
 	int& clientIndex;
 	int& accountIndex;
 	int& characterIndex;
-	CharacterMap& characterMap;
+	std::map<int, CombatData>& combatMap;
+	std::map<int, CharacterData>& characterMap;
 
 	//graphics
 	Image buttonImage;
@@ -117,15 +117,14 @@ protected:
 	Button shutDownButton;
 	//TODO: Review the camera
 	struct {
-		struct {
-			int x, y;
-		}origin, margin;
-	}camera;
-
+		int x = 0, y = 0;
+		int width = 0, height = 0;
+		int marginX = 0, marginY = 0;
+	} camera;
 	FrameRate fps;
 
 	//game
-	Character* localCharacter = nullptr;
+	CharacterData* localCharacter = nullptr;
 };
 
 #endif

client/linit.cpp

@@ -1,85 +0,0 @@
-/* $Id: linit.c,v 1.32, modified
- * Initialization of libraries for lua.c and other clients
- * See Copyright Notice in lua.h
- * 
- * If you embed Lua in your program and need to open the standard
- * libraries, call luaL_openlibs in your program. If you need a
- * different set of libraries, copy this file to your project and edit
- * it to suit your needs.
- * 
- * Modified for use in Tortuga, renamed to linit.cpp
- * Modifications are released under the zlib license:
- * 
- * Copyright: (c) Kayne Ruse 2014
- * 
- * This software is provided 'as-is', without any express or implied
- * warranty. In no event will the authors be held liable for any damages
- * arising from the use of this software.
- * 
- * Permission is granted to anyone to use this software for any purpose,
- * including commercial applications, and to alter it and redistribute it
- * freely, subject to the following restrictions:
- * 
- * 1. The origin of this software must not be misrepresented; you must not
- * claim that you wrote the original software. If you use this software
- * in a product, an acknowledgment in the product documentation would be
- * appreciated but is not required.
- * 
- * 2. Altered source versions must be plainly marked as such, and must not be
- * misrepresented as being the original software.
- * 
- * 3. This notice may not be removed or altered from any source
- * distribution.
-*/
-#define linit_c
-#define LUA_LIB
-
-#include "lua/lua.hpp"
-
-#include "region_api.hpp"
-#include "region_pager_api.hpp"
-#include "tile_sheet_api.hpp"
-
-//these libs are loaded by lua.c and are readily available to any Lua program
-static const luaL_Reg loadedlibs[] = {
-	//Standard libs
-	{"_G", luaopen_base},
-	{LUA_LOADLIBNAME, luaopen_package},
-	{LUA_COLIBNAME, luaopen_coroutine},
-	{LUA_TABLIBNAME, luaopen_table},
-	{LUA_IOLIBNAME, luaopen_io},
-	{LUA_OSLIBNAME, luaopen_os},
-	{LUA_STRLIBNAME, luaopen_string},
-	{LUA_BITLIBNAME, luaopen_bit32},
-	{LUA_MATHLIBNAME, luaopen_math},
-	{LUA_DBLIBNAME, luaopen_debug},
-
-	//Tortuga's API
-	{TORTUGA_REGION_NAME, openRegionAPI},
-	{TORTUGA_REGION_PAGER_NAME, openRegionPagerAPI},
-	{TORTUGA_TILE_SHEET_NAME, openTileSheetAPI},
-
-	{NULL, NULL}
-};
-
-
-//these libs are preloaded and must be required before used
-static const luaL_Reg preloadedlibs[] = {
-	{NULL, NULL}
-};
-
-LUALIB_API void luaL_openlibs (lua_State *L) {
-	const luaL_Reg *lib;
-	//call open functions from 'loadedlibs' and set results to global table
-	for (lib = loadedlibs; lib->func; lib++) {
-		luaL_requiref(L, lib->name, lib->func, 1);
-		lua_pop(L, 1); //remove lib
-	}
-	//add open functions from 'preloadedlibs' into 'package.preload' table
-	luaL_getsubtable(L, LUA_REGISTRYINDEX, "_PRELOAD");
-	for (lib = preloadedlibs; lib->func; lib++) {
-		lua_pushcfunction(L, lib->func);
-		lua_setfield(L, -2, lib->name);
-	}
-	lua_pop(L, 1); //remove _PRELOAD table
-}

client/lobby_menu.cpp

@@ -30,25 +30,21 @@
 //Public access members
 //-------------------------
 
-LobbyMenu::LobbyMenu(lua_State* L, UDPNetworkUtility& aNetwork):
-	lua(L),
-	network(aNetwork)
+LobbyMenu::LobbyMenu(
+	ConfigUtility* const argConfig,
+	UDPNetworkUtility* const argNetwork,
+	int* const argClientIndex,
+	int* const argAccountIndex
+	):
+	config(*argConfig),
+	network(*argNetwork),
+	clientIndex(*argClientIndex),
+	accountIndex(*argAccountIndex)
 {
-	//get the config info
-	lua_getglobal(lua, "config");
-	lua_getfield(lua, -1, "dir");
-	lua_getfield(lua, -1, "interface");
-	lua_getfield(lua, -2, "fonts");
-
-	std::string interfaceDir = lua_tostring(lua, -2);
-	std::string fontsDir = lua_tostring(lua, -1);
-
-	lua_pop(lua, 4);
-
 	//setup the utility objects
-	image.LoadSurface(interfaceDir + "button_menu.bmp");
+	image.LoadSurface(config["dir.interface"] + "button_menu.bmp");
 	image.SetClipH(image.GetClipH()/3);
-	font.LoadSurface(fontsDir + "pk_white_8.bmp");
+	font.LoadSurface(config["dir.fonts"] + "pk_white_8.bmp");
 
 	//pass the utility objects
 	search.SetImage(&image);
@@ -92,11 +88,11 @@ void LobbyMenu::FrameStart() {
 
 void LobbyMenu::Update(double delta) {
 	//suck in and process all waiting packets
-	SerialPacket* packetBuffer = new SerialPacket();
+	SerialPacket* packetBuffer = static_cast<SerialPacket*>(malloc(MAX_PACKET_SIZE));
 	while(network.Receive(packetBuffer)) {
 		HandlePacket(packetBuffer);
 	}
-	delete packetBuffer;
+	free(static_cast<void*>(packetBuffer));
 }
 
 void LobbyMenu::FrameEnd() {
@@ -152,44 +148,26 @@ void LobbyMenu::MouseButtonDown(SDL_MouseButtonEvent const& button) {
 }
 
 void LobbyMenu::MouseButtonUp(SDL_MouseButtonEvent const& button) {
-	//prep
-	lua_getglobal(lua, "config");
-
 	if (search.MouseButtonUp(button) == Button::State::HOVER) {
-		//get the set parameters
-		lua_getfield(lua, -1, "server");
-		lua_getfield(lua, -1, "host");
-		lua_getfield(lua, -2, "port");
-
 		//broadcast to the network, or a specific server
 		SerialPacket packet;
 		packet.type = SerialPacketType::BROADCAST_REQUEST;
-		network.SendTo(lua_tostring(lua, -2), lua_tointeger(lua, -1), &packet);
+		network.SendTo(config["server.host"].c_str(), config.Int("server.port"), &packet);
 
 		//reset the server list
 		serverInfo.clear();
 		selection = nullptr;
-
-		//clear the parameters
-		lua_pop(lua, 3);
 	}
 
 	else if (join.MouseButtonUp(button) == Button::State::HOVER && selection != nullptr && selection->compatible) {
-		//get the parameters
-		lua_getfield(lua, -1, "client");
-		lua_getfield(lua, -1, "username");
-
 		//pack the packet
 		ClientPacket packet;
 		packet.type = SerialPacketType::JOIN_REQUEST;
-		strncpy(packet.username, lua_tostring(lua, -1), PACKET_STRING_SIZE);
+		strncpy(packet.username, config["client.username"].c_str(), PACKET_STRING_SIZE);
 
 		//join the selected server
 		network.SendTo(&selection->address, &packet);
 		selection = nullptr;
-
-		//clear the parameters
-		lua_pop(lua, 2);
 	}
 
 	else if (back.MouseButtonUp(button) == Button::State::HOVER) {
@@ -209,9 +187,6 @@ void LobbyMenu::MouseButtonUp(SDL_MouseButtonEvent const& button) {
 	else {
 		selection = nullptr;
 	}
-
-	//clear the parameters
-	lua_pop(lua, 1);
 }
 
 void LobbyMenu::KeyDown(SDL_KeyboardEvent const& key) {
@@ -229,10 +204,10 @@ void LobbyMenu::KeyUp(SDL_KeyboardEvent const& key) {
 void LobbyMenu::HandlePacket(SerialPacket* const argPacket) {
 		switch(argPacket->type) {
 		case SerialPacketType::BROADCAST_RESPONSE:
-			HandleBroadcastResponse(dynamic_cast<ServerPacket*>(argPacket));
+			HandleBroadcastResponse(static_cast<ServerPacket*>(argPacket));
 		break;
 		case SerialPacketType::JOIN_RESPONSE:
-			HandleJoinResponse(dynamic_cast<ClientPacket*>(argPacket));
+			HandleJoinResponse(static_cast<ClientPacket*>(argPacket));
 		break;
 		//handle errors
 		default:
@@ -257,25 +232,16 @@ void LobbyMenu::HandleBroadcastResponse(ServerPacket* const argPacket) {
 }
 
 void LobbyMenu::HandleJoinResponse(ClientPacket* const argPacket) {
-	lua_getglobal(lua, "config");
-	lua_getfield(lua, -1, "client");
-	lua_getfield(lua, -1, "handle");
-	lua_getfield(lua, -2, "avatar");
-
-	lua_pushinteger(lua, argPacket->clientIndex);
-	lua_pushinteger(lua, argPacket->accountIndex);
-	lua_setfield(lua, -5, "accountIndex");
-	lua_setfield(lua, -4, "clientIndex");
+	clientIndex = argPacket->clientIndex;
+	accountIndex = argPacket->accountIndex;
 	network.Bind(&argPacket->srcAddress, Channels::SERVER);
 	SetNextScene(SceneList::INWORLD);
 
 	//send this player's character info
 	CharacterPacket newPacket;
 	newPacket.type = SerialPacketType::CHARACTER_NEW;
-	strncpy(newPacket.handle, lua_tostring(lua, -2), PACKET_STRING_SIZE);
-	strncpy(newPacket.avatar, lua_tostring(lua, -1), PACKET_STRING_SIZE);
-	newPacket.accountIndex = argPacket->accountIndex;
+	strncpy(newPacket.handle, config["client.handle"].c_str(), PACKET_STRING_SIZE);
+	strncpy(newPacket.avatar, config["client.avatar"].c_str(), PACKET_STRING_SIZE);
+	newPacket.accountIndex = accountIndex;
 	network.SendTo(Channels::SERVER, &newPacket);
-
-	lua_pop(lua, 4);
 }

client/lobby_menu.hpp

@@ -28,21 +28,24 @@
 #include "button.hpp"
 
 //utilities
+#include "config_utility.hpp"
 #include "udp_network_utility.hpp"
 
 //client
 #include "base_scene.hpp"
 
-//APIs
-#include "lua/lua.hpp"
-
 //STL
 #include <vector>
 
 class LobbyMenu : public BaseScene {
 public:
 	//Public access members
-	LobbyMenu(lua_State*, UDPNetworkUtility&);
+	LobbyMenu(
+		ConfigUtility* const argConfig,
+		UDPNetworkUtility* const argNetwork,
+		int* const argClientIndex,
+		int* const argAccountIndex
+	);
 	~LobbyMenu();
 
 protected:
@@ -65,8 +68,10 @@ protected:
 	void HandleJoinResponse(ClientPacket* const);
 
 	//shared parameters
-	lua_State* lua = nullptr;
+	ConfigUtility& config;
 	UDPNetworkUtility& network;
+	int& clientIndex;
+	int& accountIndex;
 
 	//members
 	Image image;
@@ -87,7 +92,7 @@ protected:
 	std::vector<ServerInformation> serverInfo;
 
 	//a terrible hack, forgive me
-	//TODO: I'd love a proper gui system for this
+	//I'd love a proper gui system for this
 	SDL_Rect listBox;
 	ServerInformation* selection = nullptr;
 };

client/main.cpp

@@ -27,6 +27,7 @@
 using namespace std;
 
 int main(int argc, char** argv) {
+	cout << "Beginning client" << endl;
 	try {
 		ClientApplication app;
 		app.Init(argc, argv);
@@ -37,5 +38,6 @@ int main(int argc, char** argv) {
 		cerr << "Fatal exception thrown: " << e.what() << endl;
 		return 1;
 	}
+	cout << "Clean exit" << endl;
 	return 0;
 }

client/main_menu.cpp

@@ -25,22 +25,13 @@
 //Public access members
 //-------------------------
 
-MainMenu::MainMenu(lua_State* L): lua(L) {
-	//get the config info
-	lua_getglobal(lua, "config");
-	lua_getfield(lua, -1, "dir");
-	lua_getfield(lua, -1, "interface");
-	lua_getfield(lua, -2, "fonts");
-
-	std::string interfaceDir = lua_tostring(lua, -2);
-	std::string fontsDir = lua_tostring(lua, -1);
-
-	lua_pop(lua, 4);
-
+MainMenu::MainMenu(ConfigUtility* const argConfig):
+	config(*argConfig)
+{
 	//setup the utility objects
-	image.LoadSurface(interfaceDir + "button_menu.bmp");
+	image.LoadSurface(config["dir.interface"] + "button_menu.bmp");
 	image.SetClipH(image.GetClipH()/3);
-	font.LoadSurface(fontsDir + "pk_white_8.bmp");
+	font.LoadSurface(config["dir.fonts"] + "pk_white_8.bmp");
 
 	//pass the utility objects
 	startButton.SetImage(&image);
@@ -111,13 +102,12 @@ void MainMenu::MouseButtonDown(SDL_MouseButtonEvent const& button) {
 
 void MainMenu::MouseButtonUp(SDL_MouseButtonEvent const& button) {
 	if (startButton.MouseButtonUp(button) == Button::State::HOVER) {
-		SetNextScene(SceneList::INWORLD);
+		SetNextScene(SceneList::LOBBYMENU);
 	}
 	if (optionsButton.MouseButtonUp(button) == Button::State::HOVER) {
 		SetNextScene(SceneList::OPTIONSMENU);
 	}
-	if (quitButton.MouseButtonUp(button) == Button::State::HOVER &&
-		button.button & SDL_BUTTON_LMASK) {
+	if (quitButton.MouseButtonUp(button) == Button::State::HOVER) {
 		QuitEvent();
 	}
 }

client/main_menu.hpp

@@ -24,16 +24,15 @@
 
 #include "base_scene.hpp"
 
+#include "config_utility.hpp"
 #include "image.hpp"
 #include "raster_font.hpp"
 #include "button.hpp"
 
-#include "lua/lua.hpp"
-
 class MainMenu : public BaseScene {
 public:
 	//Public access members
-	MainMenu(lua_State* L);
+	MainMenu(ConfigUtility* const);
 	~MainMenu();
 
 protected:
@@ -51,7 +50,7 @@ protected:
 	void KeyUp(SDL_KeyboardEvent const&);
 
 	//shared parameters
-	lua_State* lua = nullptr;
+	ConfigUtility& config;
 
 	//members
 	Image image;

client/makefile

@@ -1,7 +1,7 @@
 #config
-INCLUDES+=. scenes ../common/debugging ../common/gameplay ../common/graphics ../common/map ../common/network ../common/network/packet ../common/network/serial ../common/ui ../common/utilities
-LIBS+=client.a ../libcommon.a -lSDL_net -lwsock32 -liphlpapi -lmingw32 -lSDLmain -lSDL -llua
-CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
+INCLUDES+=. ../common/gameplay ../common/graphics ../common/map ../common/network ../common/network/packet ../common/network/serial ../common/ui ../common/utilities
+LIBS+=../libcommon.a -lSDL_net -lwsock32 -liphlpapi -lmingw32 -lSDLmain -lSDL -llua
+CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES)) -DGRAPHICS
 
 #source
 CXXSRC=$(wildcard *.cpp)
@@ -16,7 +16,6 @@ OUT=$(addprefix $(OUTDIR)/,client)
 
 #targets
 all: $(OBJ) $(OUT)
-	$(MAKE) -C scenes
 	$(CXX) $(CXXFLAGS) -o $(OUT) $(OBJ) $(LIBS)
 
 $(OBJ): | $(OBJDIR)

client/options_menu.cpp

@@ -25,22 +25,13 @@
 //Public access members
 //-------------------------
 
-OptionsMenu::OptionsMenu(lua_State* L): lua(L) {
-	//get the config info
-	lua_getglobal(lua, "config");
-	lua_getfield(lua, -1, "dir");
-	lua_getfield(lua, -1, "interface");
-	lua_getfield(lua, -2, "fonts");
-
-	std::string interfaceDir = lua_tostring(lua, -2);
-	std::string fontsDir = lua_tostring(lua, -1);
-
-	lua_pop(lua, 4);
-
+OptionsMenu::OptionsMenu(ConfigUtility* const argConfig):
+	config(*argConfig)
+{
 	//setup the utility objects
-	image.LoadSurface(interfaceDir + "button_menu.bmp");
+	image.LoadSurface(config["dir.interface"] + "button_menu.bmp");
 	image.SetClipH(image.GetClipH()/3);
-	font.LoadSurface(fontsDir + "pk_white_8.bmp");
+	font.LoadSurface(config["dir.fonts"] + "pk_white_8.bmp");
 
 	//pass the utility objects
 	backButton.SetImage(&image);
@@ -93,8 +84,7 @@ void OptionsMenu::MouseButtonDown(SDL_MouseButtonEvent const& button) {
 }
 
 void OptionsMenu::MouseButtonUp(SDL_MouseButtonEvent const& button) {
-	if (backButton.MouseButtonUp(button) == Button::State::HOVER &&
-		button.button & SDL_BUTTON_LMASK) {
+	if (backButton.MouseButtonUp(button) == Button::State::HOVER) {
 		SetNextScene(SceneList::MAINMENU);
 	}
 }

client/options_menu.hpp

@@ -24,17 +24,16 @@
 
 #include "base_scene.hpp"
 
+#include "config_utility.hpp"
 #include "image.hpp"
 #include "raster_font.hpp"
 #include "button.hpp"
 
-#include "lua/lua.hpp"
-
 //TODO: The options screen needs to be USED
 class OptionsMenu : public BaseScene {
 public:
 	//Public access members
-	OptionsMenu(lua_State* L);
+	OptionsMenu(ConfigUtility* const);
 	~OptionsMenu();
 
 protected:
@@ -52,7 +51,7 @@ protected:
 	void KeyUp(SDL_KeyboardEvent const&);
 
 	//shared parameters
-	lua_State* lua = nullptr;
+	ConfigUtility& config;
 
 	//members
 	Image image;

client/splash_screen.cpp

@@ -21,21 +21,14 @@
 */
 #include "splash_screen.hpp"
 
-#include <string>
-
 //-------------------------
 //Public access members
 //-------------------------
 
-SplashScreen::SplashScreen(lua_State* L): lua(L) {
-	//get the config info
-	lua_getglobal(lua, "config");
-	lua_getfield(lua, -1, "dir");
-	lua_getfield(lua, -1, "logos");
-	std::string logos = lua_tostring(lua, -1);
-	lua_pop(lua, 3);
-
-	logo.LoadSurface(logos + "krstudios.bmp");
+SplashScreen::SplashScreen(ConfigUtility* const argConfig):
+	config(*argConfig)
+{
+	logo.LoadSurface(config["dir.logos"] + "krstudios.bmp");
 	startTick = std::chrono::steady_clock::now();
 }
 

client/splash_screen.hpp

@@ -24,16 +24,15 @@
 
 #include "base_scene.hpp"
 
+#include "config_utility.hpp"
 #include "image.hpp"
 
-#include "lua/lua.hpp"
-
 #include <chrono>
 
 class SplashScreen : public BaseScene {
 public:
 	//Public access members
-	SplashScreen(lua_State* L);
+	SplashScreen(ConfigUtility* const);
 	~SplashScreen();
 
 protected:
@@ -42,7 +41,7 @@ protected:
 	void Render(SDL_Surface* const);
 
 	//shared parameters
-	lua_State* lua = nullptr;
+	ConfigUtility& config;
 
 	//members
 	std::chrono::steady_clock::time_point startTick;

common/gameplay/character_data.cpp

@@ -19,23 +19,27 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#include "character.hpp"
+#include "character_data.hpp"
 
-void Character::Update(double delta) {
+void CharacterData::Update(double delta) {
 	if (motion.x && motion.y) {
 		origin += motion * delta * CHARACTER_WALKING_MOD;
 	}
 	else if (motion != 0) {
 		origin += motion * delta;
 	}
+#ifdef GRAPHICS
 	sprite.Update(delta);
+#endif
 }
 
-void Character::DrawTo(SDL_Surface* const dest, int camX, int camY) {
+#ifdef GRAPHICS
+
+void CharacterData::DrawTo(SDL_Surface* const dest, int camX, int camY) {
 	sprite.DrawTo(dest, origin.x - camX, origin.y - camY);
 }
 
-void Character::CorrectSprite() {
+void CharacterData::CorrectSprite() {
 	//NOTE: These must correspond to the sprite sheet in use
 	if (motion.y > 0) {
 		sprite.SetYIndex(0);
@@ -59,3 +63,5 @@ void Character::CorrectSprite() {
 		sprite.SetXIndex(0);
 	}
 }
+
+#endif

common/gameplay/character_data.hpp

@@ -22,15 +22,22 @@
 #ifndef CHARACTERDATA_HPP_
 #define CHARACTERDATA_HPP_
 
-//components
-#include "character_defines.hpp"
 #include "vector2.hpp"
 #include "statistics.hpp"
 
+//graphics
+#ifdef GRAPHICS
+ #include "sprite_sheet.hpp"
+#endif
+
 //std namespace
 #include <string>
 #include <cmath>
 
+//the speeds that the characters move
+constexpr double CHARACTER_WALKING_SPEED = 140.0;
+constexpr double CHARACTER_WALKING_MOD = 1.0/sqrt(2.0);
+
 struct CharacterData {
 	//metadata
 	int owner;
@@ -41,14 +48,25 @@ struct CharacterData {
 	int roomIndex = 0;
 	Vector2 origin = {0.0,0.0};
 	Vector2 motion = {0.0,0.0};
+	Vector2 bounds = {0.0,0.0};
 
 	//base statistics
 	Statistics stats;
 
 	//TODO: gameplay components: equipment, items, buffs, debuffs
 
+	//methods
+	void Update(double delta);
+#ifdef GRAPHICS
+	void DrawTo(SDL_Surface* const, int camX, int camY);
+	void CorrectSprite();
+#endif
+
 	//active gameplay members
 	//NOTE: these are lost when unloaded
+#ifdef GRAPHICS
+	SpriteSheet sprite;
+#endif
 	bool inCombat = false;
 	int atbGauge = 0;
 	//TODO: stored command

common/gameplay/combat_data.hpp

@@ -23,18 +23,31 @@
 #define COMBATDATA_HPP_
 
 #include "vector2.hpp"
-#include "combat_defines.hpp"
 
 //gameplay members
 #include "character_data.hpp"
 #include "enemy_data.hpp"
 
+//graphics
+#ifdef GRAPHICS
+ #include "sprite_sheet.hpp"
+#endif
+
 //std namespace
 #include <chrono>
 #include <array>
 #include <utility>
 
+#define COMBAT_MAX_CHARACTERS 12
+#define COMBAT_MAX_ENEMIES 12
+
 struct CombatData {
+	enum class Terrain {
+		//TODO: types of combat terrains
+		NONE = 0,
+		GRASSLANDS,
+	};
+
 	typedef std::chrono::steady_clock Clock;
 
 	std::array<CharacterData, COMBAT_MAX_CHARACTERS> characterArray;
@@ -47,6 +60,11 @@ struct CombatData {
 
 	//time interval
 	Clock::time_point lastTick = Clock::now();
+
+	//graphics
+#ifdef GRAPHICS
+	SpriteSheet sprite;
+#endif
 };
 
 #endif

common/gameplay/enemy_data.hpp

@@ -25,6 +25,11 @@
 #include "vector2.hpp"
 #include "statistics.hpp"
 
+//graphics
+#ifdef GRAPHICS
+ #include "sprite_sheet.hpp"
+#endif
+
 //std namespace
 #include <string>
 
@@ -40,6 +45,11 @@ struct EnemyData {
 
 	//active gameplay members
 	//NOTE: these are lost when unloaded
+#ifdef GRAPHICS
+	SpriteSheet sprite;
+	Vector2 origin = {0.0,0.0};
+	Vector2 bounds = {0.0,0.0};
+#endif
 	int tableIndex;
 	int atbGauge = 0;
 };

common/gameplay/makefile

@@ -1,7 +1,7 @@
 #config
-INCLUDES+=.
+INCLUDES+=. ../utilities ../graphics
 LIBS+=
-CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
+CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES)) -DGRAPHICS
 
 #source
 CXXSRC=$(wildcard *.cpp)

common/gameplay/sanity_check.cpp

@@ -0,0 +1,31 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "character_data.hpp"
+#include "combat_data.hpp"
+#include "enemy_data.hpp"
+#include "statistics.hpp"
+
+/* DOCS: Sanity check, read more
+ * Since most/all of the files in this directory are header files, I've created
+ * this source file as a "sanity check", to ensure that the above header files
+ * are written correctly via make.
+*/

common/graphics/tile_sheet.cpp

@@ -21,24 +21,24 @@
 */
 #include "tile_sheet.hpp"
 
-void TileSheet::Load(std::string fname, int tileWidth, int tileHeight) {
+void TileSheet::Load(std::string fname, int xc, int yc) {
+	XCount = xc;
+	YCount = yc;
 	image.LoadSurface(fname);
-	image.SetClipW(tileWidth);
-	image.SetClipH(tileHeight);
-	xCount = image.GetSurface()->w / image.GetClipW();
-	yCount = image.GetSurface()->h / image.GetClipH();
+	image.SetClipW(image.GetClipW()/XCount);
+	image.SetClipH(image.GetClipH()/YCount);
 }
 
 void TileSheet::Unload() {
 	image.FreeSurface();
-	xCount = yCount = 0;
+	XCount = YCount = 0;
 }
 
-void TileSheet::DrawTileTo(SDL_Surface* const dest, int x, int y, Region::type_t tile) {
+void TileSheet::DrawTo(SDL_Surface* const dest, int x, int y, Region::type_t tile) {
 	//0 is invisible
 	if (tile == 0) return;
-	image.SetClipX((tile-1) % xCount * image.GetClipW());
-	image.SetClipY((tile-1) / xCount * image.GetClipH());
+	image.SetClipX((tile-1) % XCount * image.GetClipW());
+	image.SetClipY((tile-1) / XCount * image.GetClipH());
 	image.DrawTo(dest, x, y);
 }
 
@@ -50,8 +50,8 @@ void TileSheet::DrawRegionTo(SDL_Surface* const dest, Region* const region, int
 				tile = region->GetTile(i, j, k);
 				//0 is invisible
 				if (tile == 0) continue;
-				image.SetClipX((tile-1) % xCount * image.GetClipW());
-				image.SetClipY((tile-1) / xCount * image.GetClipH());
+				image.SetClipX((tile-1) % XCount * image.GetClipW());
+				image.SetClipY((tile-1) / XCount * image.GetClipH());
 				image.DrawTo(dest,
 					(region->GetX() + i) * image.GetClipW() - camX,
 					(region->GetY() + j) * image.GetClipH() - camY);

common/graphics/tile_sheet.hpp

@@ -31,24 +31,24 @@
 class TileSheet {
 public:
 	TileSheet() = default;
-	TileSheet(std::string f, int w, int h) { Load(f, w, h); }
+	TileSheet(std::string f, int x, int y) { Load(f, x, y); }
 	~TileSheet() = default;
 
-	void Load(std::string fname, int tileWidth, int tileHeight);
+	void Load(std::string fname, int XCount, int YCount);
 	void Unload();
 
-	void DrawTileTo(SDL_Surface* const dest, int x, int y, Region::type_t tile);
+	void DrawTo(SDL_Surface* const dest, int x, int y, Region::type_t tile);
 	void DrawRegionTo(SDL_Surface* const dest, Region* const region, int camX, int camY);
 
 	//accessors
 	Image* GetImage() { return ℑ }
-	int GetXCount() { return xCount; }
-	int GetYCount() { return yCount; }
+	int GetXCount() { return XCount; }
+	int GetYCount() { return YCount; }
 	int GetTileW() { return image.GetClipW(); }
 	int GetTileH() { return image.GetClipH(); }
 private:
 	Image image;
-	int xCount = 0, yCount = 0;
+	int XCount = 0, YCount = 0;
 };
 
 #endif

common/makefile

@@ -1,5 +1,4 @@
 all:
-	$(MAKE) -C debugging
 	$(MAKE) -C gameplay
 	$(MAKE) -C graphics
 	$(MAKE) -C map

common/map/makefile

@@ -1,5 +1,5 @@
 #config
-INCLUDES+=. ../utilities ../graphics
+INCLUDES+=. ../utilities
 LIBS+=
 CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
 

common/map/region.cpp

@@ -35,7 +35,6 @@ Region::Region(int argX, int argY): x(argX), y(argY) {
 
 Region::Region(Region const& rhs): x(rhs.x), y(rhs.y) {
 	memcpy(tiles, rhs.tiles, REGION_WIDTH*REGION_HEIGHT*REGION_DEPTH*sizeof(type_t));
-	solid = rhs.solid;
 }
 
 Region::type_t Region::SetTile(int x, int y, int z, type_t v) {
@@ -45,11 +44,3 @@ Region::type_t Region::SetTile(int x, int y, int z, type_t v) {
 Region::type_t Region::GetTile(int x, int y, int z) {
 	return tiles[x][y][z];
 }
-
-bool Region::SetSolid(int x, int y, bool b) {
-	return solid[x * REGION_WIDTH + y] = b;
-}
-
-bool Region::GetSolid(int x, int y) {
-	return solid[x * REGION_WIDTH + y];
-}

common/map/region.hpp

@@ -22,17 +22,10 @@
 #ifndef REGION_HPP_
 #define REGION_HPP_
 
-#include <bitset>
-#include <cmath>
-
-//the region's storage format
 constexpr int REGION_WIDTH = 20;
 constexpr int REGION_HEIGHT = 20;
 constexpr int REGION_DEPTH = 3;
 
-//the size of the solid map
-constexpr int REGION_SOLID_FOOTPRINT = ceil(REGION_WIDTH * REGION_HEIGHT / 8.0);
-
 class Region {
 public:
 	typedef unsigned char type_t;
@@ -45,23 +38,14 @@ public:
 	type_t SetTile(int x, int y, int z, type_t v);
 	type_t GetTile(int x, int y, int z);
 
-	bool SetSolid(int x, int y, bool b);
-	bool GetSolid(int x, int y);
-
 	//accessors
 	int GetX() const { return x; }
 	int GetY() const { return y; }
-
-	std::bitset<REGION_WIDTH*REGION_HEIGHT>* GetSolidBitset() { return &solid; }
 private:
 	const int x;
 	const int y;
 
 	type_t tiles[REGION_WIDTH][REGION_HEIGHT][REGION_DEPTH];
-	std::bitset<REGION_WIDTH*REGION_HEIGHT> solid;
 };
 
-//the memory footprint of the tile and solid data; not including any metadata
-constexpr int REGION_FOOTPRINT = REGION_WIDTH * REGION_HEIGHT * REGION_DEPTH * sizeof(Region::type_t) + REGION_SOLID_FOOTPRINT;
-
 #endif

common/map/region_api.cpp

@@ -37,20 +37,6 @@ static int getTile(lua_State* L) {
 	return 1;
 }
 
-static int setSolid(lua_State* L) {
-	Region* region = reinterpret_cast<Region*>(lua_touserdata(L, 1));
-	bool ret = region->SetSolid(lua_tointeger(L, 2)-1, lua_tointeger(L, 3)-1, lua_toboolean(L, 4));
-	lua_pushboolean(L, ret);
-	return 1;
-}
-
-static int getSolid(lua_State* L) {
-	Region* region = reinterpret_cast<Region*>(lua_touserdata(L, 1));
-	bool ret = region->GetSolid(lua_tointeger(L, 2)-1, lua_tointeger(L, 3)-1);
-	lua_pushboolean(L, ret);
-	return 1;
-}
-
 static int getX(lua_State* L) {
 	Region* region = reinterpret_cast<Region*>(lua_touserdata(L, 1));
 	lua_pushinteger(L, region->GetX());
@@ -78,41 +64,39 @@ static int getDepth(lua_State* L) {
 	return 1;
 }
 
-static int load(lua_State* L) {
-	//EMPTY
+static int onLoad(lua_State* L) {
+	//TODO: onLoad()
 	lua_pushboolean(L, false);
 	return 1;
 }
 
-static int save(lua_State* L) {
-	//EMPTY
+static int onSave(lua_State* L) {
+	//TODO: onSave()
 	return 0;
 }
 
-static int create(lua_State* L) {
-	//EMPTY
+static int onCreate(lua_State* L) {
+	//TODO: onCreate()
 	return 0;
 }
 
-static int unload(lua_State* L) {
-	//EMPTY
+static int onUnload(lua_State* L) {
+	//TODO: onUnload()
 	return 0;
 }
 
 static const luaL_Reg regionLib[] = {
 	{"SetTile",setTile},
 	{"GetTile",getTile},
-	{"SetSolid",setSolid},
-	{"GetSolid",getSolid},
 	{"GetX",getX},
 	{"GetY",getY},
 	{"GetWidth",getWidth},
 	{"GetHeight",getHeight},
 	{"GetDepth",getDepth},
-	{"Load",load},
-	{"Save",save},
-	{"Create",create},
-	{"Unload",unload},
+	{"OnLoad",onLoad},
+	{"OnSave",onSave},
+	{"OnCreate",onCreate},
+	{"OnUnload",onUnload},
 	{nullptr, nullptr}
 };
 

common/map/region_pager_api.cpp

@@ -24,14 +24,10 @@
 #include "region_pager_lua.hpp"
 #include "region.hpp"
 
+#include <stdexcept>
 #include <string>
 
-static int getRegionPager(lua_State* L) {
-	lua_pushstring(L, TORTUGA_REGION_PAGER_PSEUDO_INDEX);
-	lua_gettable(L, LUA_REGISTRYINDEX);
-}
-
-//DOCS: These glue functions simply wrap RegionPagerLua's methods
+//DOCS: These functions are just wrappers for the RegionPagerLua class
 
 static int setTile(lua_State* L) {
 	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
@@ -47,20 +43,6 @@ static int getTile(lua_State* L) {
 	return 1;
 }
 
-static int setSolid(lua_State* L) {
-	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
-	bool ret = pager->SetSolid(lua_tointeger(L, 2), lua_tointeger(L, 3), lua_toboolean(L, 4));
-	lua_pushboolean(L, ret);
-	return 1;
-}
-
-static int getSolid(lua_State* L) {
-	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
-	bool ret = pager->GetSolid(lua_tointeger(L, 2), lua_tointeger(L, 3));
-	lua_pushboolean(L, ret);
-	return 1;
-}
-
 static int getRegion(lua_State* L) {
 	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
 	Region* region = pager->GetRegion(lua_tointeger(L, 2), lua_tointeger(L, 3));
@@ -68,6 +50,20 @@ static int getRegion(lua_State* L) {
 	return 1;
 }
 
+static int setDirectory(lua_State* L) {
+	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
+	std::string s = pager->SetDirectory(lua_tostring(L, 2));
+	lua_pushstring(L, s.c_str());
+	return 1;
+}
+
+static int getDirectory(lua_State* L) {
+	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
+	std::string s = pager->GetDirectory();
+	lua_pushstring(L, s.c_str());
+	return 1;
+}
+
 static int loadRegion(lua_State* L) {
 	RegionPagerLua* pager = reinterpret_cast<RegionPagerLua*>(lua_touserdata(L, 1));
 	Region* region = pager->LoadRegion(lua_tointeger(L, 2), lua_tointeger(L, 3));
@@ -95,13 +91,12 @@ static int unloadRegion(lua_State* L) {
 	return 0;
 }
 
-static const luaL_Reg regionPagerLib[] = {
-	{"GetRegionPager", getRegionPager},
+static const luaL_Reg pagerlib[] = {
 	{"SetTile", setTile},
 	{"GetTile", getTile},
-	{"SetSolid", setSolid},
-	{"GetSolid", getSolid},
 	{"GetRegion", getRegion},
+	{"SetDirectory", setDirectory},
+	{"GetDirectory", getDirectory},
 	{"LoadRegion", loadRegion},
 	{"SaveRegion", saveRegion},
 	{"CreateRegion", createRegion},
@@ -110,6 +105,6 @@ static const luaL_Reg regionPagerLib[] = {
 };
 
 LUAMOD_API int openRegionPagerAPI(lua_State* L) {
-	luaL_newlib(L, regionPagerLib);
+	luaL_newlib(L, pagerlib);
 	return 1;
 }

common/map/region_pager_api.hpp

@@ -19,12 +19,11 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#ifndef REGIONPAGERAPI_HPP_
-#define REGIONPAGERAPI_HPP_
+#ifndef PAGERAPI_HPP_
+#define PAGERAPI_HPP_
 
 #include "lua/lua.hpp"
 
-#define TORTUGA_REGION_PAGER_PSEUDO_INDEX "RegionPagerPseudoIndex"
 #define TORTUGA_REGION_PAGER_NAME "RegionPager"
 LUAMOD_API int openRegionPagerAPI(lua_State* L);
 

common/map/region_pager_base.cpp

@@ -36,20 +36,7 @@ Region::type_t RegionPagerBase::GetTile(int x, int y, int z) {
 	return ptr->GetTile(x - ptr->GetX(), y - ptr->GetY(), z);
 }
 
-bool RegionPagerBase::SetSolid(int x, int y, int b) {
-	Region* ptr = GetRegion(x, y);
-	return ptr->SetSolid(x - ptr->GetX(), y - ptr->GetY(), b);
-}
-
-bool RegionPagerBase::GetSolid(int x, int y) {
-	Region* ptr = GetRegion(x, y);
-	return ptr->GetSolid(x - ptr->GetX(), y - ptr->GetY());
-}
-
 Region* RegionPagerBase::GetRegion(int x, int y) {
-	x = snapToBase(REGION_WIDTH, x);
-	y = snapToBase(REGION_HEIGHT, y);
-
 	//get the region by various means
 	Region* ptr = nullptr;
 	ptr = FindRegion(x, y);

common/map/region_pager_base.hpp

@@ -35,10 +35,6 @@ public:
 	virtual Region::type_t SetTile(int x, int y, int z, Region::type_t v);
 	virtual Region::type_t GetTile(int x, int y, int z);
 
-	//solid manipulation
-	virtual bool SetSolid(int x, int y, int b);
-	virtual bool GetSolid(int x, int y);
-
 	//region manipulation
 	virtual Region* GetRegion(int x, int y);
 	virtual Region* FindRegion(int x, int y);

common/map/region_pager_lua.cpp

@@ -32,18 +32,19 @@ Region* RegionPagerLua::LoadRegion(int x, int y) {
 	Region tmpRegion(x, y);
 
 	//API hook
-	lua_getglobal(lua, "Region");
-	lua_getfield(lua, -1, "Load");
-	lua_pushlightuserdata(lua, &tmpRegion);
-	if (lua_pcall(lua, 1, 1, 0) != LUA_OK) {
-		throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(lua, -1) ));
+	lua_getglobal(luaState, "Region");
+	lua_getfield(luaState, -1, "OnLoad");
+	lua_pushlightuserdata(luaState, &tmpRegion);
+	lua_pushstring(luaState, directory.c_str());
+	if (lua_pcall(luaState, 2, 1, 0) != LUA_OK) {
+		throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(luaState, -1) ));
 	}
 	//success or failure
-	if (!lua_toboolean(lua, -1)) {
-		lua_pop(lua, 2);
+	if (!lua_toboolean(luaState, -1)) {
+		lua_pop(luaState, 2);
 		return nullptr;
 	}
-	lua_pop(lua, 2);
+	lua_pop(luaState, 2);
 	regionList.push_front(tmpRegion);
 	return &regionList.front();
 }
@@ -53,13 +54,14 @@ Region* RegionPagerLua::SaveRegion(int x, int y) {
 	Region* ptr = FindRegion(x, y);
 	if (ptr) {
 		//API hook
-		lua_getglobal(lua, "Region");
-		lua_getfield(lua, -1, "Save");
-		lua_pushlightuserdata(lua, ptr);
-		if (lua_pcall(lua, 1, 0, 0) != LUA_OK) {
-			throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(lua, -1) ));
+		lua_getglobal(luaState, "Region");
+		lua_getfield(luaState, -1, "OnSave");
+		lua_pushlightuserdata(luaState, ptr);
+		lua_pushstring(luaState, directory.c_str());
+		if (lua_pcall(luaState, 2, 0, 0) != LUA_OK) {
+			throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(luaState, -1) ));
 		}
-		lua_pop(lua, 1);
+		lua_pop(luaState, 1);
 	}
 	return ptr;
 }
@@ -73,29 +75,30 @@ Region* RegionPagerLua::CreateRegion(int x, int y) {
 	Region tmpRegion(x, y);
 
 	//API hook
-	lua_getglobal(lua, "Region");
-	lua_getfield(lua, -1, "Create");
-	lua_pushlightuserdata(lua, &tmpRegion);
+	lua_getglobal(luaState, "Region");
+	lua_getfield(luaState, -1, "OnCreate");
+	lua_pushlightuserdata(luaState, &tmpRegion);
 	//TODO: parameters
-	if (lua_pcall(lua, 1, 0, 0) != LUA_OK) {
-		throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(lua, -1) ));
+	if (lua_pcall(luaState, 1, 0, 0) != LUA_OK) {
+		throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(luaState, -1) ));
 	}
-	lua_pop(lua, 1);
+	lua_pop(luaState, 1);
 	regionList.push_front(tmpRegion);
 	return &regionList.front();
 }
 
 void RegionPagerLua::UnloadRegion(int x, int y) {
-	lua_getglobal(lua, "Region");
+	lua_getglobal(luaState, "Region");
 
 	regionList.remove_if([&](Region& region) -> bool {
 		if (region.GetX() == x && region.GetY() == y) {
 
 			//API hook
-			lua_getfield(lua, -1, "Unload");
-			lua_pushlightuserdata(lua, &region);
-			if (lua_pcall(lua, 1, 0, 0) != LUA_OK) {
-				throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(lua, -1) ));
+			lua_getfield(luaState, -1, "OnUnload");
+			lua_pushlightuserdata(luaState, &region);
+			lua_pushstring(luaState, directory.c_str());
+			if (lua_pcall(luaState, 2, 0, 0) != LUA_OK) {
+				throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(luaState, -1) ));
 			}
 
 			return true;
@@ -103,21 +106,22 @@ void RegionPagerLua::UnloadRegion(int x, int y) {
 		return false;
 	});
 
-	lua_pop(lua, 1);
+	lua_pop(luaState, 1);
 }
 
 void RegionPagerLua::UnloadAll() {
-	lua_getglobal(lua, "Region");
+	lua_getglobal(luaState, "Region");
 
 	for (auto& it : regionList) {
 		//API hook
-		lua_getfield(lua, -1, "Unload");
-		lua_pushlightuserdata(lua, &it);
-		if (lua_pcall(lua, 1, 0, 0) != LUA_OK) {
-			throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(lua, -1) ));
+		lua_getfield(luaState, -1, "OnUnload");
+		lua_pushlightuserdata(luaState, &it);
+		lua_pushstring(luaState, directory.c_str());
+		if (lua_pcall(luaState, 2, 0, 0) != LUA_OK) {
+			throw(std::runtime_error(std::string() + "Lua error: " + lua_tostring(luaState, -1) ));
 		}
 	}
 
-	lua_pop(lua, 1);
+	lua_pop(luaState, 1);
 	regionList.clear();
 }

common/map/region_pager_lua.hpp

@@ -42,10 +42,14 @@ public:
 	void UnloadAll() override;
 
 	//accessors & mutators
-	lua_State* SetLuaState(lua_State* L) { return lua = L; }
-	lua_State* GetLuaState() { return lua; }
+	std::string SetDirectory(std::string s) { return directory = s; }
+	std::string GetDirectory() { return directory; }
+
+	lua_State* SetLuaState(lua_State* L) { return luaState = L; }
+	lua_State* GetLuaState() { return luaState; }
 protected:
-	lua_State* lua = nullptr;
+	std::string directory;
+	lua_State* luaState = nullptr;
 };
 
 #endif

common/map/tile_sheet_api.cpp

@@ -1,82 +0,0 @@
-/* Copyright: (c) Kayne Ruse 2014
- * 
- * This software is provided 'as-is', without any express or implied
- * warranty. In no event will the authors be held liable for any damages
- * arising from the use of this software.
- * 
- * Permission is granted to anyone to use this software for any purpose,
- * including commercial applications, and to alter it and redistribute it
- * freely, subject to the following restrictions:
- * 
- * 1. The origin of this software must not be misrepresented; you must not
- * claim that you wrote the original software. If you use this software
- * in a product, an acknowledgment in the product documentation would be
- * appreciated but is not required.
- * 
- * 2. Altered source versions must be plainly marked as such, and must not be
- * misrepresented as being the original software.
- * 
- * 3. This notice may not be removed or altered from any source
- * distribution.
-*/
-#include "tile_sheet_api.hpp"
-
-#include "tile_sheet.hpp"
-
-static int getTileSheet(lua_State* L) {
-	lua_pushstring(L, TORTUGA_TILE_SHEET_PSEUDO_INDEX);
-	lua_gettable(L, LUA_REGISTRYINDEX);
-	return 1;
-}
-
-static int load(lua_State* L) {
-	TileSheet* sheet = reinterpret_cast<TileSheet*>(lua_touserdata(L, 1));
-	sheet->Load(lua_tostring(L, 2), lua_tointeger(L, 3), lua_tointeger(L, 4));
-	return 0;
-}
-
-static int unload(lua_State* L) {
-	TileSheet* sheet = reinterpret_cast<TileSheet*>(lua_touserdata(L, 1));
-	sheet->Unload();
-	return 0;
-}
-
-static int getXCount(lua_State* L) {
-	TileSheet* sheet = reinterpret_cast<TileSheet*>(lua_touserdata(L, 1));
-	lua_pushinteger(L, sheet->GetXCount());
-	return 1;
-}
-
-static int getYCount(lua_State* L) {
-	TileSheet* sheet = reinterpret_cast<TileSheet*>(lua_touserdata(L, 1));
-	lua_pushinteger(L, sheet->GetYCount());
-	return 1;
-}
-
-static int getTileW(lua_State* L) {
-	TileSheet* sheet = reinterpret_cast<TileSheet*>(lua_touserdata(L, 1));
-	lua_pushinteger(L, sheet->GetTileW());
-	return 1;
-}
-
-static int getTileH(lua_State* L) {
-	TileSheet* sheet = reinterpret_cast<TileSheet*>(lua_touserdata(L, 1));
-	lua_pushinteger(L, sheet->GetTileH());
-	return 1;
-}
-
-static const luaL_Reg tileSheetLib[] = {
-	{"GetTileSheet",getTileSheet},
-	{"Load",load},
-	{"Unload",unload},
-	{"GetXCount",getXCount},
-	{"GetYCount",getYCount},
-	{"GetTileW",getTileW},
-	{"GetTileH",getTileH},
-	{nullptr, nullptr}
-};
-
-LUAMOD_API int openTileSheetAPI(lua_State* L) {
-	luaL_newlib(L, tileSheetLib);
-	return 1;
-}

common/network/packet/combat_packet.hpp

@@ -24,13 +24,13 @@
 
 #include "serial_packet_base.hpp"
 
-#include "combat_defines.hpp"
+#include "combat_data.hpp"
 
 struct CombatPacket : SerialPacketBase {
 	//identify the combat instance
 	int combatIndex;
 	int difficulty;
-	TerrainType terrainType;
+	CombatData::Terrain terrainType;
 
 	//combatants
 	int characterArray[COMBAT_MAX_CHARACTERS];

common/network/packet/serial_packet_base.hpp

@@ -30,7 +30,7 @@
 
 #include "SDL/SDL_net.h"
 
-constexpr int NETWORK_VERSION = 20140701;
+constexpr int NETWORK_VERSION = 20140607;
 constexpr int PACKET_STRING_SIZE = 100;
 
 struct SerialPacketBase {

common/network/serial/serial.hpp

@@ -53,15 +53,13 @@ void deserializeRegionContent(RegionPacket*, void*);
 void deserializeServer(ServerPacket*, void*);
 void deserializeStatistics(Statistics*, void*);
 
-/* DOCS: Keep PACKET_BUFFER_SIZE up to date
- * DOCS: SerialPacketType::REGION_CONTENT is currently the largest type of packet, read more
- * The metadata used are:
- *     SerialPacketType
- *     room index
- *     X & Y positon
- * The rest is taken up by the Regions's content.
+/* DOCS: Keep the PACKET_BUFFER_SIZE up to date
+ * DOCS: REGION_CONTENT is currently the largest type of packet, read more
+ *  map content: REGION_WIDTH * REGION_HEIGHT * REGION_DEPTH * sizoeof(region::type_t)
+ *  map format: sizeof(int) * 3
+ *  metadata: sizeof(SerialPacket::Type)
 */
 
-constexpr int PACKET_BUFFER_SIZE = sizeof(SerialPacketType) + sizeof(int) * 3 + REGION_FOOTPRINT;
+constexpr int PACKET_BUFFER_SIZE = REGION_WIDTH * REGION_HEIGHT * REGION_DEPTH * sizeof(Region::type_t) + sizeof(int) * 3 + sizeof(SerialPacketType);
 
 #endif

common/network/serial/serial_combat.cpp

@@ -29,7 +29,7 @@ void serializeCombat(CombatPacket* packet, void* buffer) {
 	//identify the combat instance
 	SERIALIZE(buffer, &packet->combatIndex, sizeof(int));
 	SERIALIZE(buffer, &packet->difficulty, sizeof(int));
-	SERIALIZE(buffer, &packet->terrainType, sizeof(TerrainType));
+	SERIALIZE(buffer, &packet->terrainType, sizeof(CombatData::Terrain));
 
 	//combatants
 	SERIALIZE(buffer, &packet->characterArray, sizeof(int) * COMBAT_MAX_CHARACTERS);
@@ -49,7 +49,7 @@ void deserializeCombat(CombatPacket* packet, void* buffer) {
 	//identify the combat instance
 	DESERIALIZE(buffer, &packet->combatIndex, sizeof(int));
 	DESERIALIZE(buffer, &packet->difficulty, sizeof(int));
-	DESERIALIZE(buffer, &packet->terrainType, sizeof(TerrainType));
+	DESERIALIZE(buffer, &packet->terrainType, sizeof(CombatData::Terrain));
 
 	//combatants
 	DESERIALIZE(buffer, &packet->characterArray, sizeof(int) * COMBAT_MAX_CHARACTERS);

common/network/serial/serial_region.cpp

@@ -40,7 +40,7 @@ void serializeRegionContent(RegionPacket* packet, void* buffer) {
 	SERIALIZE(buffer, &packet->x, sizeof(int));
 	SERIALIZE(buffer, &packet->y, sizeof(int));
 
-	//tiles
+	//content
 	for (register int i = 0; i < REGION_WIDTH; i++) {
 		for (register int j = 0; j < REGION_HEIGHT; j++) {
 			for (register int k = 0; k < REGION_DEPTH; k++) {
@@ -49,9 +49,6 @@ void serializeRegionContent(RegionPacket* packet, void* buffer) {
 			}
 		}
 	}
-
-	//solids
-	SERIALIZE(buffer, packet->region->GetSolidBitset(), REGION_SOLID_FOOTPRINT);
 }
 
 void deserializeRegionFormat(RegionPacket* packet, void* buffer) {
@@ -74,7 +71,7 @@ void deserializeRegionContent(RegionPacket* packet, void* buffer) {
 	//an object to work on
 	packet->region = new Region(packet->x, packet->y);
 
-	//tiles
+	//content
 	for (register int i = 0; i < REGION_WIDTH; i++) {
 		for (register int j = 0; j < REGION_HEIGHT; j++) {
 			for (register int k = 0; k < REGION_DEPTH; k++) {
@@ -83,7 +80,4 @@ void deserializeRegionContent(RegionPacket* packet, void* buffer) {
 			}
 		}
 	}
-
-	//solids
-	DESERIALIZE(buffer, packet->region->GetSolidBitset(), REGION_SOLID_FOOTPRINT);
 }

common/ui/menu_bar.hpp

@@ -36,7 +36,6 @@
  * This class needs a rewrite.
 */
 
-//TODO: This thing is fucking terrible, fix it and the button class
 class MenuBar {
 public:
 	MenuBar() = default;

common/utilities/bounding_box.hpp

@@ -1,77 +0,0 @@
-/* Copyright: (c) Kayne Ruse 2013, 2014
- * 
- * This software is provided 'as-is', without any express or implied
- * warranty. In no event will the authors be held liable for any damages
- * arising from the use of this software.
- * 
- * Permission is granted to anyone to use this software for any purpose,
- * including commercial applications, and to alter it and redistribute it
- * freely, subject to the following restrictions:
- * 
- * 1. The origin of this software must not be misrepresented; you must not
- * claim that you wrote the original software. If you use this software
- * in a product, an acknowledgment in the product documentation would be
- * appreciated but is not required.
- * 
- * 2. Altered source versions must be plainly marked as such, and must not be
- * misrepresented as being the original software.
- * 
- * 3. This notice may not be removed or altered from any source
- * distribution.
-*/
-#ifndef BOUNDINGBOX_HPP_
-#define BOUNDINGBOX_HPP_
-
-#include <type_traits>
-#include <algorithm>
-
-class BoundingBox {
-public:
-	//This is explicitly a POD
-	int x, y;
-	int w, h;
-
-	BoundingBox() = default;
-	BoundingBox(int i, int j, int k, int l): x(i), y(j), w(k), h(l) {};
-	~BoundingBox() = default;
-	BoundingBox& operator=(BoundingBox const&) = default;
-
-	int Size() {
-		return std::max(w*h,0);
-	}
-
-	bool CheckOverlap(BoundingBox rhs) {
-		return !(
-			x >= rhs.x + rhs.w ||
-			y >= rhs.y + rhs.h ||
-			rhs.x >= x + w ||
-			rhs.y >= y + h);
-	}
-
-	BoundingBox CalcOverlap(BoundingBox rhs) {
-		if (!CheckOverlap(rhs)) {
-			return {0, 0, 0, 0};
-		}
-		BoundingBox ret;
-		ret.x = std::max(x, rhs.x);
-		ret.y = std::max(y, rhs.y);
-		ret.w = std::min(x+w, rhs.x+rhs.w) - ret.x;
-		ret.h = std::min(y+h, rhs.y+rhs.h) - ret.y;
-		return ret;
-	}
-};
-
-//This is explicitly a POD
-static_assert(std::is_pod<Vector2>::value, "BoundingBox is not a POD");
-
-#include "vector2.hpp"
-
-//operators
-inline BoundingBox operator+(BoundingBox b, Vector2 v) {
-	return {b.x + (int)v.x, b.y + (int)v.y, b.w, b.h};
-}
-inline BoundingBox operator+(Vector2 v, BoundingBox b) {
-	return b + v;
-}
-
-#endif

common/utilities/check_bounds.cpp

@@ -0,0 +1,40 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "check_bounds.hpp"
+
+bool checkPoint(Vector2 const& origin, Vector2 const& bound, Vector2 const& point) {
+	return !(
+		point.x < origin.x ||
+		point.y < origin.y ||
+		point.x >= origin.x + bound.x ||
+		point.y >= origin.y + bound.y
+	);
+}
+
+bool checkOverlap(Vector2 const& originOne, Vector2 const& boundOne, Vector2 const& originTwo, Vector2 const& boundTwo) {
+	return !(
+		originOne.x >= originTwo.x + boundTwo.x ||
+		originOne.x + boundOne.x >= originTwo.x ||
+		originOne.y >= originTwo.y + boundTwo.y ||
+		originOne.y + boundOne.y >= originTwo.y
+	);
+}

common/utilities/check_bounds.hpp

@@ -0,0 +1,30 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef CHECKBOUNDS_HPP_
+#define CHECKBOUNDS_HPP_
+
+#include "vector2.hpp"
+
+bool checkPoint(Vector2 const& origin, Vector2 const& bound, Vector2 const& point);
+bool checkOverlap(Vector2 const& originOne, Vector2 const& boundOne, Vector2 const& originTwo, Vector2 const& boundTwo);
+
+#endif

common/utilities/config_utility.cpp

@@ -0,0 +1,105 @@
+/* Copyright: (c) Kayne Ruse 2013, 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "config_utility.hpp"
+
+#include <cstdlib>
+#include <fstream>
+#include <stdexcept>
+
+using namespace std;
+
+void ConfigUtility::Load(string fname) {
+	ifstream is(fname);
+
+	if (!is.is_open()) {
+		throw(runtime_error("Failed to open config file"));
+	}
+
+	string key, val;
+
+	for (;;) { //forever
+		//eat whitespace
+		while(isspace(is.peek()))
+			is.ignore();
+
+		//end of file
+		if (is.eof())
+			break;
+
+		//skip comment lines
+		if (is.peek() == '#') {
+			while(is.peek() != '\n' && !is.eof()) {
+				is.ignore();
+			}
+			continue;
+		}
+
+		//read in the pair
+		getline(is, key,'=');
+		getline(is, val);
+
+		//trim the strings at the start & end
+		while(key.size() && isspace(*key.begin())) key.erase(0, 1);
+		while(val.size() && isspace(*val.begin())) val.erase(0, 1);
+
+		while(key.size() && isspace(*(key.end()-1))) key.erase(key.end() - 1);
+		while(val.size() && isspace(*(val.end()-1))) val.erase(val.end() - 1);
+
+		//allow empty/wiped values
+		if (key.size() == 0) {
+			continue;
+		}
+
+		//save the pair
+		table[key] = val;
+	}
+
+	is.close();
+}
+
+std::string& ConfigUtility::String(std::string s) {
+	return table[s];
+}
+
+int ConfigUtility::Integer(std::string s) {
+	std::map<std::string, std::string>::iterator it = table.find(s);
+	if (it == table.end()) {
+		return 0;
+	}
+	return atoi(it->second.c_str());
+}
+
+double ConfigUtility::Double(std::string s) {
+	std::map<std::string, std::string>::iterator it = table.find(s);
+	if (it == table.end()) {
+		return 0.0;
+	}
+	return atof(it->second.c_str());
+}
+
+bool ConfigUtility::Boolean(std::string s) {
+	std::map<std::string, std::string>::iterator it = table.find(s);
+	if (it == table.end()) {
+		return false;
+	}
+	return it->second == "true";
+}

common/utilities/config_utility.hpp

@@ -0,0 +1,60 @@
+/* Copyright: (c) Kayne Ruse 2013, 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef CONFIGUTILITY_HPP_
+#define CONFIGUTILITY_HPP_
+
+#include <map>
+#include <string>
+
+class ConfigUtility {
+public:
+	ConfigUtility() = default;
+	ConfigUtility(std::string s) { Load(s); }
+
+	void Load(std::string fname);
+
+	//convert to a type
+	std::string& String(std::string);
+	int Integer(std::string);
+	double Double(std::string);
+	bool Boolean(std::string);
+
+	//shorthand
+	std::string& operator[](std::string s) {
+		return String(s);
+	}
+	int Int(std::string s) {
+		return Integer(s);
+	}
+	bool Bool(std::string s) {
+		return Boolean(s);
+	}
+
+	//OO breaker
+	std::map<std::string, std::string>* GetMap() {
+		return &table;
+	}
+private:
+	std::map<std::string, std::string> table;
+};
+
+#endif

common/utilities/simple_rng.hpp

@@ -0,0 +1,42 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef SIMPLERNG_HPP_
+#define SIMPLERNG_HPP_
+
+//a simple, stateless, random number generator
+class SimpleRNG {
+public:
+	SimpleRNG() { SetSeed(8891.0); }
+	SimpleRNG(double x) { SetSeed(x); }
+
+	double SetSeed(double s) { return seed = s; }
+	double GetSeed() { return seed; }
+
+	double operator()(double x) {
+		return (x + seed) * 11235.0 + 81321.0;
+	};
+
+private:
+	double seed;
+};
+
+#endif

common/utilities/utility.cpp

@@ -32,10 +32,6 @@ int snapToBase(int base, int x) {
 	return x / base * base;
 }
 
-double snapToBase(double base, double x) {
-	return floor(x / base) * base;
-}
-
 std::string truncatePath(std::string pathname) {
 	return std::string(
 		std::find_if(

common/utilities/utility.hpp

@@ -25,8 +25,6 @@
 #include <string>
 
 int snapToBase(int base, int x);
-double snapToBase(double base, double x);
-
 std::string truncatePath(std::string pathname);
 
 //fixing known bugs in g++

common/utilities/vector2.hpp

@@ -43,8 +43,6 @@ public:
 	}
 	void Normalize() {
 		double l = Length();
-		if (l == 0)
-			throw(std::domain_error("Divide by zero"));
 		x /= l;
 		y /= l;
 	}

rsc/config.cfg

@@ -0,0 +1,29 @@
+#configuration of the programs
+
+#server specific settings
+server.host = 255.255.255.255
+server.port = 21795
+server.name = local
+
+server.dbname = database.db
+
+#client specific settings
+client.screen.f = false
+
+client.username = Kayne Ruse
+client.handle = Ratstail91
+client.avatar = elliot2.bmp
+
+#directories
+dir.fonts = rsc/graphics/fonts/
+dir.logos = rsc/graphics/logos/
+dir.sprites = rsc/graphics/sprites/
+dir.tilesets = rsc/graphics/tilesets/
+dir.interface = rsc/graphics/interface/
+dir.scripts = rsc/scripts/
+dir.maps = rsc/maps/
+
+#map system
+map.savename = servermap
+
+#debugging

rsc/scripts/setup_server.lua

@@ -1,45 +1,42 @@
-print("Lua script check")
+print("Lua script check (./rsc)")
 
 --uber lazy declarations
-function math.sqr(x) return x*x end
-function math.dist(x, y, i, j) return math.sqrt(math.sqr(x - i) + math.sqr(y - j)) end
+function square(x) return x*x end
+function distance(x, y, i, j) return math.sqrt(square(x - i) + square(y - j)) end
 
---objects to work on
---TODO: sheet is not accessable in the server
-local sheet = TileSheet.GetTileSheet()
-local pager = RegionPager.GetRegionPager()
-
---the selected tilesheet
-TileSheet.Load(sheet, config.dir.tilesets .. "terrain.bmp", 32, 32)
-
---tile macros, mapped to this tilesheet
+--tile macros, mapped to the tilesheet
 local base = 14
 local shift = 36
-tiles = {
-	plains	= base + shift * 0,
-	grass	= base + shift * 1,
-	dirt	= base + shift * 2,
-	sand	= base + shift * 3,
-	water	= base + shift * 4
-}
+plains	= base + shift * 0
+grass	= base + shift * 1
+dirt	= base + shift * 2
+sand	= base + shift * 3
+water	= base + shift * 4
 
---TODO: could set custom generation systems here, that differ from the global generators, etc.
---TODO: I need a way to allow for different generation algorithms for different pager objects
---TODO: This design requires only one pager, but this is not good.
-function Region.Create(region)
+--Overwrite the original OnCreate with my own version
+Region.hcOnCreate = Region.OnCreate
+Region.OnCreate = function(region)
+	local ret = Region.hcOnCreate(region) --best practices
 	for i = 1, Region.GetWidth() do
 		for j = 1, Region.GetHeight() do
-			local dist = math.dist(0, 0, i + Region.GetX(region) -1, j + Region.GetY(region) -1)
-			if dist < 10 then
-				Region.SetTile(region, i, j, 1, tiles.plains)
-			elseif dist < 12 then
-				Region.SetTile(region, i, j, 1, tiles.sand)
+			if distance(0, 0, i + Region.GetX(region) -1, j + Region.GetY(region) -1) > 10 then
+				Region.SetTile(region, i, j, 1, water)
 			else
-				Region.SetTile(region, i, j, 1, tiles.water)
-				Region.SetSolid(region, i, j, true)
+				Region.SetTile(region, i, j, 1, plains)
 			end
 		end
 	end
+	return ret
 end
 
+--Get some regions
+newRoom = RoomMgr.CreateRoom("overworld")
+pager = Room.GetPager(newRoom)
+regionTable = {
+	RegionPager.GetRegion(pager, 0, 0),
+	RegionPager.GetRegion(pager, 0, -20),
+	RegionPager.GetRegion(pager, -20, 0),
+	RegionPager.GetRegion(pager, -20, -20)
+}
+
 print("Finished the lua script")

rsc/setup.lua

@@ -1,60 +0,0 @@
---[[
-	--reroute the program while in development
-	config = {debug = true}
-	dofile("../rsc/setup.lua")
---]]
-
---catch the debug signal if the program was rerouted
-local debug = false
-if config ~= nil then
-	debug = config.debug
-end
-
---the program's configuration
-config = {
-	--common stuff
-	debug = debug,
-	dir = {
-		fonts = "rsc/graphics/fonts/",
-		logos = "rsc/graphics/logos/",
-		sprites = "rsc/graphics/sprites/",
-		tilesets = "rsc/graphics/tilesets/",
-		interface = "rsc/graphics/interface/",
-		scripts = "rsc/scripts/",
-		maps = "rsc/maps/"
-	},
-
-	--client specific stuff
-	client = {
-		screen = {
-			width = 800,
-			height = 600,
-			fullscreen = false
-		},
-		username = "Kayne",
-		handle = "Ratstail91",
-		avatar = "elliot2.bmp",
-
-		--NOTE: these generally go here
---		clientIndex
---		accountIndex
---		characterIndex
-	},
-
-	--server specific stuff
-	server = {
-		mapname = "mapsave",
-		host = "255.255.255.255",
-		port = 21795,
-		name = "local",
-		dbname = "database.db"
-	}
-}
-
---development environment
-if config.debug then
-	for k, v in pairs(config.dir) do
-		config.dir[k] = string.format("../%s", v)
-	end
-end
-

server/client_data.hpp

@@ -26,7 +26,6 @@
 
 struct ClientData {
 	IPaddress address = {0,0};
-	//TODO: ping system?
 };
 
 #endif

server/combat/makefile

@@ -1,5 +1,5 @@
 #config
-INCLUDES+=. ../../common/gameplay ../../common/utilities ../characters ../enemies
+INCLUDES+=. ../../common/gameplay ../../common/utilities
 LIBS+=
 CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
 

server/enemies/enemy_factory_generic.cpp

@@ -0,0 +1,34 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "enemy_factory_generic.hpp"
+
+EnemyFactoryGeneric::EnemyFactoryGeneric() : EnemyFactoryInterface() {
+	//EMPTY
+}
+
+EnemyFactoryGeneric::~EnemyFactoryGeneric() noexcept {
+	//EMPTY
+}
+
+void EnemyFactoryGeneric::Generate(std::list<EnemyData>* container) {
+	//TODO: fill this out
+}

server/enemies/enemy_factory_generic.hpp

@@ -0,0 +1,42 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef ENEMYFACTORYGENERIC_HPP_
+#define ENEMYFACTORYGENERIC_HPP_
+
+#include "enemy_factory_interface.hpp"
+
+#include "enemy_data.hpp"
+
+#include <list>
+
+//DOCS: Not really intended for use, but rather for copying and tweaking
+class EnemyFactoryGeneric : public EnemyFactoryInterface {
+public:
+	EnemyFactoryGeneric();
+	~EnemyFactoryGeneric() noexcept override;
+
+	void Generate(std::list<EnemyData>* container) override;
+private:
+	//TODO: hold the parameters specified by the room
+};
+
+#endif

server/enemies/enemy_factory_interface.hpp

@@ -0,0 +1,49 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef ENEMYFACTORYINTERFACE_HPP_
+#define ENEMYFACTORYINTERFACE_HPP_
+
+#include "enemy_data.hpp"
+#include "map_type.hpp"
+
+#include <list>
+
+//NOTE: Based on biome, world difficulty, etc.
+class EnemyFactoryInterface {
+public:
+	EnemyFactoryInterface() = default;
+	virtual ~EnemyFactoryInterface() = default;
+
+	virtual void Generate(std::list<EnemyData>* container) = 0;
+
+	//control the difficulty of the room
+	MapType SetType(MapType t) { return type = t; }
+	MapType GetType() { return type; }
+
+	int SetDifficulty(int d) { return difficulty = d; }
+	int GetDifficulty() { return difficulty; }
+protected:
+	MapType type;
+	int difficulty = 0;
+};
+
+#endif

server/linit.cpp

@@ -40,6 +40,7 @@
 #include "region_pager_api.hpp"
 #include "room_api.hpp"
 #include "room_mgr_api.hpp"
+#include "generator_api.hpp"
 
 //these libs are loaded by lua.c and are readily available to any Lua program
 static const luaL_Reg loadedlibs[] = {
@@ -60,6 +61,7 @@ static const luaL_Reg loadedlibs[] = {
 	{TORTUGA_REGION_PAGER_NAME, openRegionPagerAPI},
 	{TORTUGA_ROOM_NAME, openRoomAPI},
 	{TORTUGA_ROOM_MGR_NAME, openRoomMgrAPI},
+	{TORTUGA_GENRATOR_NAME, openGeneratorAPI},
 
 	{NULL, NULL}
 };

server/main.cpp

@@ -21,12 +21,15 @@
 */
 #include "server_application.hpp"
 
+#include "SDL/SDL.h"
+
 #include <stdexcept>
 #include <iostream>
 
 using namespace std;
 
 int main(int argc, char** argv) {
+	cout << "Beginning server" << endl;
 	try {
 		ServerApplication app;
 		app.Init(argc, argv);
@@ -34,8 +37,9 @@ int main(int argc, char** argv) {
 		app.Quit();
 	}
 	catch(exception& e) {
-		cerr << "Fatal exception thrown: " << e.what() << endl;
+		cerr << "Fatal error: " << e.what() << endl;
 		return 1;
 	}
+	cout << "Clean exit" << endl;
 	return 0;
 }

server/makefile

@@ -1,5 +1,5 @@
 #config
-INCLUDES+=. accounts characters combat enemies mapgen mapgen/generators rooms ../common/debugging ../common/gameplay ../common/map ../common/network ../common/network/packet ../common/network/serial ../common/utilities
+INCLUDES+=. accounts characters combat enemies mapgen mapgen/generators rooms ../common/gameplay ../common/map ../common/network ../common/network/packet ../common/network/serial ../common/utilities
 LIBS+=server.a ../libcommon.a -lSDL_net -lwsock32 -liphlpapi -lmingw32 -lSDLmain -lSDL -llua -lsqlite3
 CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
 
@@ -20,6 +20,7 @@ all: $(OBJ) $(OUT)
 	$(MAKE) -C characters
 	$(MAKE) -C combat
 	$(MAKE) -C enemies
+	$(MAKE) -C mapgen
 	$(MAKE) -C rooms
 	$(CXX) $(CXXFLAGS) -o $(OUT) $(OBJ) $(LIBS)
 

server/mapgen/chunk_data.hpp

@@ -0,0 +1,44 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef CHUNKDATA_HPP_
+#define CHUNKDATA_HPP_
+
+#include "terrain_type.hpp"
+
+#include <type_traits>
+
+struct ChunkData {
+	enum class Moddable {
+		LOCKED, //do not change
+		HARD, //minor changes
+		SOFT, //major changes
+		CLEAR, //untouched
+	};
+
+	TerrainType type;
+//	int fountainCount;
+	Moddable mod;
+};
+
+static_assert(std::is_pod<ChunkData>::value, "ChunkData is not a POD");
+
+#endif

server/mapgen/generator_api.cpp

@@ -0,0 +1,79 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "generator_api.hpp"
+
+#include "base_generator.hpp"
+
+static int getMapType(lua_State* L) {
+	BaseGenerator* ptr = reinterpret_cast<BaseGenerator*>(lua_touserdata(L, 1));
+	switch(ptr->GetMapType()) {
+		case MapType::NONE:
+			lua_pushstring(L, "none");
+		break;
+		case MapType::OVERWORLD:
+			lua_pushstring(L, "overworld");
+		break;
+		case MapType::RUINS:
+			lua_pushstring(L, "ruins");
+		break;
+		case MapType::TOWERS:
+			lua_pushstring(L, "towers");
+		break;
+		case MapType::FORESTS:
+			lua_pushstring(L, "forests");
+		break;
+		case MapType::CAVES:
+			lua_pushstring(L, "caves");
+		break;
+	}
+	return 1;
+}
+
+static int getChunk(lua_State* L) {
+	BaseGenerator* ptr = reinterpret_cast<BaseGenerator*>(lua_touserdata(L, 1));
+	ChunkData* chunk = ptr->GetChunk(lua_tointeger(L, 2), lua_tointeger(L, 3));
+	lua_pushlightuserdata(L, reinterpret_cast<void*>(chunk));
+	return 1;
+}
+
+static int getMapWidth(lua_State* L) {
+	lua_pushinteger(L, MAP_WIDTH);
+	return 1;
+}
+
+static int getMapHeight(lua_State* L) {
+	lua_pushinteger(L, MAP_HEIGHT);
+	return 1;
+}
+
+static const luaL_Reg generatorLib[] = {
+	{"GetMapType", getMapType},
+	{"GetChunk", getChunk},
+	{"GetMapWidth", getMapWidth},
+	{"GetMapHeight", getMapHeight},
+	{nullptr, nullptr}
+};
+
+LUAMOD_API int openGeneratorAPI(lua_State* L) {
+	luaL_newlib(L, generatorLib);
+	return 1;
+}

server/mapgen/generator_api.hpp

@@ -19,13 +19,12 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#ifndef TILESHEETAPI_HPP_
-#define TILESHEETAPI_HPP_
+#ifndef GENERATORAPI_HPP_
+#define GENERATORAPI_HPP_
 
 #include "lua/lua.hpp"
 
-#define TORTUGA_TILE_SHEET_PSEUDO_INDEX "TileSheetPseudoIndex"
-#define TORTUGA_TILE_SHEET_NAME "TileSheet"
-LUAMOD_API int openTileSheetAPI(lua_State* L);
+#define TORTUGA_GENRATOR_NAME "Generator"
+LUAMOD_API int openGeneratorAPI(lua_State* L);
 
 #endif

server/mapgen/generators/base_generator.cpp

@@ -0,0 +1,36 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "base_generator.hpp"
+
+BaseGenerator::BaseGenerator(MapType t) {
+	mapType = t;
+	for (int i = 0; i < MAP_WIDTH; i++) {
+		for (int j = 0; j < MAP_HEIGHT; j++) {
+			chunks[i][j].type = TerrainType::NONE;
+			chunks[i][j].mod = ChunkData::Moddable::CLEAR;
+		}
+	}
+}
+
+BaseGenerator::~BaseGenerator() {
+	//
+}

server/mapgen/generators/base_generator.hpp

@@ -19,36 +19,36 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#ifndef TIMER_HPP_
-#define TIMER_HPP_
+#ifndef BASEGENERATOR_HPP_
+#define BASEGENERATOR_HPP_
 
-#include <chrono>
-#include <string>
-#include <ostream>
+#include "map_type.hpp"
+#include "chunk_data.hpp"
 
-class Timer {
+#include "lua/lua.hpp"
+
+constexpr int MAP_WIDTH = 256;
+constexpr int MAP_HEIGHT = 256;
+
+class BaseGenerator {
 public:
-	typedef std::chrono::high_resolution_clock Clock;
-
-	Timer() = default;
-	Timer(std::string s) : name(s), start(Clock::now()) {};
-	~Timer() = default;
-
-	inline void Start() { start = Clock::now(); }
-	inline void Stop() { time = Clock::now() - start; }
+	virtual ~BaseGenerator();
 
 	//accessors and mutators
-	Clock::duration GetTime() { return time; }
+	virtual ChunkData* GetChunk(int x, int y) { return &chunks[x][y]; }
 
-	std::string SetName(std::string s) { return name = s; }
-	std::string GetName() { return name; }
+	MapType GetMapType() { return mapType; }
 
-private:
-	std::string name;
-	Clock::time_point start;
-	Clock::duration time;
+	lua_State* SetLuaState(lua_State* L) { return luaState = L; }
+	lua_State* GetLuaState() { return luaState; }
+
+protected:
+	BaseGenerator() = delete;
+	BaseGenerator(MapType t);
+
+	ChunkData chunks[MAP_WIDTH][MAP_HEIGHT];
+	MapType mapType = MapType::NONE;
+	lua_State* luaState = nullptr;
 };
 
-std::ostream& operator<<(std::ostream& os, Timer& t);
-
 #endif

server/mapgen/generators/caves_generator.cpp

@@ -19,11 +19,12 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#include "timer.hpp"
+#include "caves_generator.hpp"
 
-std::ostream& operator<<(std::ostream& os, Timer& t) {
-	os << t.GetName() << ": ";
-	os << std::chrono::duration_cast<std::chrono::nanoseconds>(t.GetTime()).count();
-	os << "ns";
-	return os;
+CavesGenerator::CavesGenerator() : BaseGenerator(MapType::CAVES) {
+	//
+}
+
+CavesGenerator::~CavesGenerator() {
+	//
 }

server/mapgen/generators/caves_generator.hpp

@@ -19,17 +19,18 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#ifndef CHARACTERDEFINES_HPP_
-#define CHARACTERDEFINES_HPP_
+#ifndef CAVESGENERATOR_HPP_
+#define CAVESGENERATOR_HPP_
 
-#include <cmath>
+#include "base_generator.hpp"
 
-//the speeds that the characters move
-constexpr double CHARACTER_WALKING_SPEED = 140.0;
-constexpr double CHARACTER_WALKING_MOD = 1.0/sqrt(2.0);
+class CavesGenerator : public BaseGenerator {
+public:
+	CavesGenerator();
+	~CavesGenerator();
 
-//the bounding boxes for the characters
-constexpr double CHARACTER_BOUNDS_WIDTH = 32.0;
-constexpr double CHARACTER_BOUNDS_HEIGHT = 32.0;
+private:
+	//
+};
 
 #endif

server/mapgen/generators/forests_generator.cpp

@@ -0,0 +1,30 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "forests_generator.hpp"
+
+ForestsGenerator::ForestsGenerator() : BaseGenerator(MapType::FORESTS) {
+	//
+}
+
+ForestsGenerator::~ForestsGenerator() {
+	//
+}

server/mapgen/generators/forests_generator.hpp

@@ -0,0 +1,36 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef FORESTSGENERATOR_HPP_
+#define FORESTSGENERATOR_HPP_
+
+#include "base_generator.hpp"
+
+class ForestsGenerator : public BaseGenerator {
+public:
+	ForestsGenerator();
+	~ForestsGenerator();
+
+private:
+	//
+};
+
+#endif

server/mapgen/generators/makefile

@@ -1,5 +1,5 @@
 #config
-INCLUDES+=.
+INCLUDES+=. ..
 LIBS+=
 CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
 
@@ -11,8 +11,8 @@ OBJDIR=obj
 OBJ+=$(addprefix $(OBJDIR)/,$(CXXSRC:.cpp=.o))
 
 #output
-OUTDIR=..
-OUT=$(addprefix $(OUTDIR)/,libcommon.a)
+OUTDIR=../..
+OUT=$(addprefix $(OUTDIR)/,server.a)
 
 #targets
 all: $(OBJ) $(OUT)

server/mapgen/generators/overworld_generator.cpp

@@ -0,0 +1,30 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "overworld_generator.hpp"
+
+OverworldGenerator::OverworldGenerator() : BaseGenerator(MapType::OVERWORLD) {
+	//
+}
+
+OverworldGenerator::~OverworldGenerator() {
+	//
+}

server/mapgen/generators/overworld_generator.hpp

@@ -0,0 +1,36 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef OVERWORLDGENERATOR_HPP_
+#define OVERWORLDGENERATOR_HPP_
+
+#include "base_generator.hpp"
+
+class OverworldGenerator : public BaseGenerator {
+public:
+	OverworldGenerator();
+	~OverworldGenerator();
+
+private:
+	//
+};
+
+#endif

server/mapgen/generators/ruins_generator.cpp

@@ -0,0 +1,30 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "ruins_generator.hpp"
+
+RuinsGenerator::RuinsGenerator() : BaseGenerator(MapType::RUINS) {
+	//
+}
+
+RuinsGenerator::~RuinsGenerator() {
+	//
+}

server/mapgen/generators/ruins_generator.hpp

@@ -0,0 +1,36 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef RUINSGENERATOR_HPP_
+#define RUINSGENERATOR_HPP_
+
+#include "base_generator.hpp"
+
+class RuinsGenerator : public BaseGenerator {
+public:
+	RuinsGenerator();
+	~RuinsGenerator();
+
+private:
+	//
+};
+
+#endif

server/mapgen/generators/towers_generator.cpp

@@ -0,0 +1,30 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#include "towers_generator.hpp"
+
+TowersGenerator::TowersGenerator() : BaseGenerator(MapType::TOWERS) {
+	//
+}
+
+TowersGenerator::~TowersGenerator() {
+	//
+}

server/mapgen/generators/towers_generator.hpp

@@ -0,0 +1,36 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef TOWERSGENERATOR_HPP_
+#define TOWERSGENERATOR_HPP_
+
+#include "base_generator.hpp"
+
+class TowersGenerator : public BaseGenerator {
+public:
+	TowersGenerator();
+	~TowersGenerator();
+
+private:
+	//
+};
+
+#endif

server/mapgen/makefile

@@ -1,5 +1,5 @@
 #config
-INCLUDES+=. .. ../../common/gameplay ../../common/graphics ../../common/map ../../common/network ../../common/network/packet ../../common/network/serial ../../common/ui ../../common/utilities
+INCLUDES+=. generators
 LIBS+=
 CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
 
@@ -12,11 +12,12 @@ OBJ+=$(addprefix $(OBJDIR)/,$(CXXSRC:.cpp=.o))
 
 #output
 OUTDIR=..
-OUT=$(addprefix $(OUTDIR)/,client.a)
+OUT=$(addprefix $(OUTDIR)/,server.a)
 
 #targets
 all: $(OBJ) $(OUT)
 	ar -crs $(OUT) $(OBJ)
+	$(MAKE) -C generators
 
 $(OBJ): | $(OBJDIR)
 

server/mapgen/map_type.hpp

@@ -0,0 +1,34 @@
+/* Copyright: (c) Kayne Ruse 2014
+ * 
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ * 
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ * 
+ * 1. The origin of this software must not be misrepresented; you must not
+ * claim that you wrote the original software. If you use this software
+ * in a product, an acknowledgment in the product documentation would be
+ * appreciated but is not required.
+ * 
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ * misrepresented as being the original software.
+ * 
+ * 3. This notice may not be removed or altered from any source
+ * distribution.
+*/
+#ifndef MAPTYPE_HPP_
+#define MAPTYPE_HPP_
+
+enum class MapType {
+	NONE,
+	OVERWORLD,
+	RUINS,
+	TOWERS,
+	FORESTS,
+	CAVES,
+};
+
+#endif

server/mapgen/terrain_type.hpp

@@ -19,16 +19,22 @@
  * 3. This notice may not be removed or altered from any source
  * distribution.
 */
-#ifndef COMBATDEFINES_HPP_
-#define COMBATDEFINES_HPP_
-
-#define COMBAT_MAX_CHARACTERS 16
-#define COMBAT_MAX_ENEMIES 16
+#ifndef TERRAINTYPE_HPP_
+#define TERRAINTYPE_HPP_
 
 enum class TerrainType {
+	//default: something's wrong
 	NONE = 0,
-	GRASSLANDS,
-	//etc.
+
+	//standard overworld
+	PLAINS,
+	GRASS,
+	DIRT,
+	SAND,
+	WATER,
+
+	//not used
+	LAST,
 };
 
 #endif

server/rooms/room_api.cpp

@@ -29,6 +29,12 @@ static int getPager(lua_State* L) {
 	return 1;
 }
 
+static int getGenerator(lua_State* L) {
+	RoomData* room = reinterpret_cast<RoomData*>(lua_touserdata(L, 1));
+	lua_pushlightuserdata(L, reinterpret_cast<void*>(room->generator));
+	return 1;
+}
+
 static int onCreate(lua_State* L) {
 	//TODO: onCreate()
 	return 0;
@@ -43,6 +49,7 @@ static int onUnload(lua_State* L) {
 
 static const luaL_Reg roomLib[] = {
 	{"GetPager",getPager},
+	{"GetGenerator",getGenerator},
 	{"OnCreate", onCreate},
 	{"OnUnload", onUnload},
 	{nullptr, nullptr}

server/rooms/room_data.hpp

@@ -23,11 +23,14 @@
 #define ROOMDATA_HPP_
 
 //map system
+#include "map_type.hpp"
 #include "region_pager_lua.hpp"
+#include "base_generator.hpp"
 
 struct RoomData {
 	//members
 	RegionPagerLua pager;
+	BaseGenerator* generator = nullptr;
 
 	//TODO: collision map
 	//TODO: NPCs?

server/rooms/room_manager.cpp

@@ -21,19 +21,41 @@
 */
 #include "room_manager.hpp"
 
+//the generator types
+#include "overworld_generator.hpp"
+#include "ruins_generator.hpp"
+#include "towers_generator.hpp"
+#include "forests_generator.hpp"
+#include "caves_generator.hpp"
+
 #include <stdexcept>
 
 //-------------------------
 //public access methods
 //-------------------------
 
-RoomData* RoomManager::CreateRoom() {
+RoomData* RoomManager::CreateRoom(MapType mapType) {
 	//create the room
 	RoomData* newRoom = new RoomData();
 
+	//create the generator, use a lambda because I'm lazy
+	newRoom->generator = [mapType]() -> BaseGenerator* {
+		switch(mapType) {
+			//BUG: Not having a map type results in an overworld generator by default
+			case MapType::NONE:
+			case MapType::OVERWORLD: return new OverworldGenerator();
+			case MapType::RUINS: return new RuinsGenerator();
+			case MapType::TOWERS: return new TowersGenerator();
+			case MapType::FORESTS: return new ForestsGenerator();
+			case MapType::CAVES: return new CavesGenerator();
+		}
+		throw(std::runtime_error("Failed to set the room's generator"));
+	}();
+
 	//set the state
 	if (luaState) {
 		newRoom->pager.SetLuaState(luaState);
+		newRoom->generator->SetLuaState(luaState);
 	}
 
 	//register the room
@@ -69,13 +91,15 @@ void RoomManager::UnloadRoom(int uid) {
 	lua_pop(luaState, 1);
 
 	//free the memory
+	delete room->generator;
 	delete room;
 	roomMap.erase(uid);
 }
 
 RoomData* RoomManager::GetRoom(int uid) {
-	return FindRoom(uid);
-	//TODO: expand this to auto-create the room
+	RoomData* ptr = FindRoom(uid);
+	if (ptr) return ptr;
+	return CreateRoom(MapType::NONE);
 }
 
 RoomData* RoomManager::FindRoom(int uid) {

server/rooms/room_manager.hpp

@@ -36,7 +36,7 @@ public:
 	~RoomManager() = default;
 
 	//public access methods
-	RoomData* CreateRoom();
+	RoomData* CreateRoom(MapType);
 	void UnloadRoom(int uid);
 
 	RoomData* GetRoom(int uid);

server/rooms/room_mgr_api.cpp

@@ -46,8 +46,18 @@ static int createRoom(lua_State* L) {
 	lua_gettable(L, LUA_REGISTRYINDEX);
 	RoomManager* roomMgr = reinterpret_cast<RoomManager*>(lua_touserdata(L, -1));
 
+	//determine the specified room type
+	MapType mapType = [L]() -> MapType {
+		if (std::string("overworld") == lua_tostring(L, -2)) return MapType::OVERWORLD;
+		if (std::string("ruins") == lua_tostring(L, -2)) return MapType::RUINS;
+		if (std::string("towers") == lua_tostring(L, -2)) return MapType::TOWERS;
+		if (std::string("forests") == lua_tostring(L, -2)) return MapType::FORESTS;
+		if (std::string("caves") == lua_tostring(L, -2)) return MapType::CAVES;
+		return MapType::NONE;
+	}();
+
 	//create the room
-	RoomData* newRoom = roomMgr->CreateRoom();
+	RoomData* newRoom = roomMgr->CreateRoom(mapType);
 
 	//return the new room
 	lua_pushlightuserdata(L, newRoom);

server/server_application.cpp

@@ -21,11 +21,8 @@
 */
 #include "server_application.hpp"
 
-//for PACKET_BUFFER_SIZE
-#include "serial.hpp"
-
-//utility functions
 #include "sql_utility.hpp"
+#include "serial.hpp"
 #include "utility.hpp"
 
 #include <stdexcept>
@@ -38,9 +35,9 @@
 
 void ServerApplication::Init(int argc, char** argv) {
 	//NOTE: I might need to rearrange the init process so that lua & SQL can interact with the map system as needed.
-	std::cout << "Beginning " << argv[0] << std::endl;
+	std::cout << "Beginning startup" << std::endl;
 
-	//load the prerequisites
+	//initial setup
 	config.Load("rsc\\config.cfg");
 
 	//-------------------------
@@ -83,7 +80,9 @@ void ServerApplication::Init(int argc, char** argv) {
 	accountMgr.SetDatabase(database);
 	characterMgr.SetDatabase(database);
 
+	combatMgr.SetLuaState(luaState);
 	roomMgr.SetLuaState(luaState);
+	enemyMgr.SetLuaState(luaState);
 
 	std::cout << "Internal managers set" << std::endl;
 
@@ -114,22 +113,12 @@ void ServerApplication::Init(int argc, char** argv) {
 	//debug output
 	//-------------------------
 
-	//TODO: enable/disable these with a switch
-#define DEBUG_OUTPUT_VAR(x) std::cout << "\t" << #x << ": " << x << std::endl;
-
 	std::cout << "Internal sizes:" << std::endl;
-
-	DEBUG_OUTPUT_VAR(sizeof(Region::type_t));
-	DEBUG_OUTPUT_VAR(sizeof(Region));
-	DEBUG_OUTPUT_VAR(REGION_WIDTH);
-	DEBUG_OUTPUT_VAR(REGION_HEIGHT);
-	DEBUG_OUTPUT_VAR(REGION_DEPTH);
-	DEBUG_OUTPUT_VAR(REGION_SOLID_FOOTPRINT);
-	DEBUG_OUTPUT_VAR(REGION_FOOTPRINT);
-	DEBUG_OUTPUT_VAR(PACKET_BUFFER_SIZE);
-	DEBUG_OUTPUT_VAR(MAX_PACKET_SIZE);
-
-#undef DEBUG_OUTPUT_VAR
+	std::cout << "\tTile Size: " << sizeof(Region::type_t) << std::endl;
+	std::cout << "\tRegion Format: " << REGION_WIDTH << ", " << REGION_HEIGHT << ", " << REGION_DEPTH << std::endl;
+	std::cout << "\tRegion Content Footprint: " << REGION_WIDTH * REGION_HEIGHT * REGION_DEPTH * sizeof(Region::type_t) << std::endl;
+	std::cout << "\tPACKET_BUFFER_SIZE: " << PACKET_BUFFER_SIZE << std::endl;
+	std::cout << "\tMAX_PACKET_SIZE: " << MAX_PACKET_SIZE << std::endl;
 
 	//-------------------------
 	//finalize the startup
@@ -152,8 +141,7 @@ void ServerApplication::Proc() {
 			HandlePacket(packetBuffer);
 		}
 		//update the internals
-		//BUG: #30 Update the internals i.e. player positions
-
+		//TODO: update the internals i.e. player positions
 		//give the computer a break
 		SDL_Delay(10);
 	}
@@ -178,7 +166,7 @@ void ServerApplication::Quit() {
 	SDLNet_Quit();
 	SDL_Quit();
 
-	std::cout << "Clean exit" << std::endl;
+	std::cout << "Shutdown finished" << std::endl;
 }
 
 //-------------------------
@@ -281,16 +269,6 @@ void ServerApplication::HandleJoinRequest(ClientPacket* const argPacket) {
 
 void ServerApplication::HandleDisconnect(ClientPacket* const argPacket) {
 	//TODO: authenticate who is disconnecting/kicking
-	/*Pseudocode:
-	if sender's account index -> client index -> address == sender's address then
-		continue
-	end
-	if sender's account index -> admin == true OR sender's account index -> mod == true then
-		continue
-	end
-	if neither of the above is true, then output a warning to the console, and return
-	*/
-
 
 	//forward to the specified client
 	network.SendTo(
@@ -318,12 +296,6 @@ void ServerApplication::HandleDisconnect(ClientPacket* const argPacket) {
 
 void ServerApplication::HandleShutdown(SerialPacket* const argPacket) {
 	//TODO: authenticate who is shutting the server down
-	/*Pseudocode:
-	if sender's account -> admin is not true then
-		print a warning
-		return
-	end
-	*/
 
 	//end the server
 	running = false;
@@ -355,6 +327,12 @@ void ServerApplication::HandleRegionRequest(RegionPacket* const argPacket) {
 	network.SendTo(&argPacket->srcAddress, static_cast<SerialPacket*>(&newPacket));
 }
 
+//-------------------------
+//combat management
+//-------------------------
+
+//TODO: combat management
+
 //-------------------------
 //Character Management
 //-------------------------
@@ -421,7 +399,15 @@ void ServerApplication::HandleCharacterUpdate(CharacterPacket* const argPacket)
 		return;
 	}
 
-	//accept client-side logic
+	/* TODO: rewrite this design flaw, read more
+	 * Slaving the client to the server here is a terrible idea, instead there
+	 * needs to be a utility function to update and send the server-side character
+	 * to the clients.
+	 *
+	 * Other things to consider include functionality to reequip the character,
+	 * apply status effects and to change the stats as well. These should all be
+	 * handled server-side.
+	*/
 	character->roomIndex = argPacket->roomIndex;
 	character->origin = argPacket->origin;
 	character->motion = argPacket->motion;
@@ -433,6 +419,12 @@ void ServerApplication::HandleCharacterUpdate(CharacterPacket* const argPacket)
 	PumpPacket(argPacket);
 }
 
+//-------------------------
+//enemy management
+//-------------------------
+
+//TODO: enemy management
+
 //-------------------------
 //mismanagement
 //-------------------------

server/server_application.hpp

@@ -22,10 +22,12 @@
 #ifndef SERVERAPPLICATION_HPP_
 #define SERVERAPPLICATION_HPP_
 
-//server specific stuff, mostly managers
-#include "client_data.hpp"
+//server specific stuff
 #include "account_manager.hpp"
 #include "character_manager.hpp"
+#include "client_data.hpp"
+#include "combat_manager.hpp"
+#include "enemy_manager.hpp"
 #include "room_manager.hpp"
 
 //common utilities
@@ -65,11 +67,17 @@ private:
 	//map management
 	void HandleRegionRequest(RegionPacket* const);
 
+	//combat management
+	//TODO: combat management
+
 	//character management
 	void HandleCharacterNew(CharacterPacket* const);
 	void HandleCharacterDelete(CharacterPacket* const);
 	void HandleCharacterUpdate(CharacterPacket* const);
 
+	//enemy management
+	//TODO: enemy management
+
 	//mismanagement
 	void HandleSynchronize(ClientPacket* const);
 
@@ -91,6 +99,8 @@ private:
 	//managers
 	AccountManager accountMgr;
 	CharacterManager characterMgr;
+	CombatManager combatMgr;
+	EnemyManager enemyMgr;
 	RoomManager roomMgr;
 
 	//misc