diff RatPageController.lua @ 13:6cb9a2936580

Miscellanous Lua code consistency improvements: - no semicolon except between statements on same line - use of implicit cast to bool in if/while conditions, instead of various eq/neq against true, false or nil - no parenthesis around if/while conditions (C-ism) - avoid long function calls in if conditions - removed space in comma-separated expressions lists in multiple assignments - added spaces between arguments of functions calls - use tabs for indentation (in Lua files only) - don't reverse == in if conditions, like "if 42==foo then" (C-ism) - removed some extra parenthesis in complex expressions (C-ism) - added spaces around operators in most expressions for ease of reading - added comma after last element of table initializers - removed space after # operator - moved comment prefix of disabled code into tab (to keep disabled code aligned)
author madcatzinc@35b17cf1-18cd-47ff-9ca3-31d6b526ef09
date Thu, 25 Apr 2013 01:29:45 +0000
parents d186f8cd5000
children 80192bc4a108
line wrap: on
line diff
--- a/RatPageController.lua	Tue Apr 16 15:19:23 2013 +0000
+++ b/RatPageController.lua	Thu Apr 25 01:29:45 2013 +0000
@@ -1,4 +1,4 @@
---~ Warcraft Plugin for Cyborg MMO7 
+--~ Warcraft Plugin for Cyborg MMO7
 --~ Filename: RatPageController.lua
 --~ Description: Controller logic for the RatPage
 --~ Copyright (C) 2012 Mad Catz Inc.
@@ -20,67 +20,64 @@
 
 CyborgMMO_RatPageController = {
 	new = function()
-		local self = {};
-		CyborgMMO_RatPageModel.Instance().SetMode(1);
+		local self = {}
+		CyborgMMO_RatPageModel.Instance().SetMode(1)
 
 		self.SlotClicked = function(slot)
 			local slotObject = nil
 			slotObject = CyborgMMO_RatPageModel.Instance().GetObjectOnButton(slot.Id)
-			CyborgMMO_RatPageModel.Instance().SetObjectOnButton(slot.Id, CyborgMMO_RatPageModel.Instance().GetMode(), self.GetCursorObject());
+			CyborgMMO_RatPageModel.Instance().SetObjectOnButton(slot.Id, CyborgMMO_RatPageModel.Instance().GetMode(), self.GetCursorObject())
 
-			if(slotObject ~= nil) then
-				slotObject.Pickup();
+			if slotObject then
+				slotObject.Pickup()
 			end
-
 		end
 
 		self.ModeClicked = function(mode)
-			msg("Setting mode "..tostring(mode.Id));
-			CyborgMMO_RatPageModel.Instance().SetMode(mode.Id);
+			msg("Setting mode "..tostring(mode.Id))
+			CyborgMMO_RatPageModel.Instance().SetMode(mode.Id)
 		end
 
 		self.GetCursorObject = function()
-			local cursorObject = nil;
-			if(nil ~= GetCursorInfo()) then
-				local type, detail, subdetail = GetCursorInfo();
-				cursorObject = CyborgMMO_WowObject.Create(type, detail, subdetail);
-				ClearCursor();
+			local cursorObject = nil
+			if GetCursorInfo() then
+				local type,detail,subdetail = GetCursorInfo()
+				cursorObject = CyborgMMO_WowObject.Create(type, detail, subdetail)
+				ClearCursor()
 			end
-			return cursorObject;
+			return cursorObject
 		end
-		
+
 		self.CallbackDropped = function(callbackObject)
-			local slot = nil;
-			local observers = CyborgMMO_RatPageModel.Instance().GetAllObservers();
-			for i = 1, (# observers) do
-				if(MouseIsOver(observers[i])) then
-					slot = observers[i];
-					break;
+			local slot = nil
+			local observers = CyborgMMO_RatPageModel.Instance().GetAllObservers()
+			for i=1,#observers do
+				if MouseIsOver(observers[i]) then
+					slot = observers[i]
+					break
 				end
 			end
-			if(nil ~= slot) then 
-				CyborgMMO_RatPageModel.Instance().SetObjectOnButton(slot.Id, CyborgMMO_RatPageModel.Instance().GetMode(), callbackObject.wowObject);
+			if slot then
+				CyborgMMO_RatPageModel.Instance().SetObjectOnButton(slot.Id, CyborgMMO_RatPageModel.Instance().GetMode(), callbackObject.wowObject)
 			end
 		end
-		
-		
 
 		self.Close = function()
-
 		end
 
 		self.Open = function()
 		end
 
-		return self;
+		return self
 	end,
 
 	m_Instance = nil,
 
 	Instance = function()
-		if(nil == CyborgMMO_RatPageController.m_Instance) then
-			CyborgMMO_RatPageController.m_Instance = CyborgMMO_RatPageController.new();
+		if not CyborgMMO_RatPageController.m_Instance then
+			CyborgMMO_RatPageController.m_Instance = CyborgMMO_RatPageController.new()
 		end
-		return CyborgMMO_RatPageController.m_Instance;
-	end
+		return CyborgMMO_RatPageController.m_Instance
+	end,
 }
+