diff RatPageModel.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 8428fa7cf0e4
children 9f2d838d4f8e
line wrap: on
line diff
--- a/RatPageModel.lua	Tue Apr 16 15:19:23 2013 +0000
+++ b/RatPageModel.lua	Thu Apr 25 01:29:45 2013 +0000
@@ -1,4 +1,4 @@
---~ Warcraft Plugin for Cyborg MMO7 
+--~ Warcraft Plugin for Cyborg MMO7
 --~ Filename: RatPageModel.lua
 --~ Description: Code model of the MMO7 mouse
 --~ Copyright (C) 2012 Mad Catz Inc.
@@ -19,102 +19,104 @@
 --~ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
 -- Constants --
 
-local RAT7 = { BUTTONS = 13, MODES = 3, SHIFT = 0}
+local RAT7 = {
+	BUTTONS = 13,
+	MODES = 3,
+	SHIFT = 0,
+}
 
-local MIDDLEMOUSE = 1;
+local MIDDLEMOUSE = 1
 
 CyborgMMO_RatPageModel = {
 	new = function()
-	local self = {}
-		self.m_Mode = 1;
-		self.ObserverCount = 0;
+		local self = {}
+		self.m_Mode = 1
+		self.ObserverCount = 0
 		self.ObserverList = {}
 		self.Data = {}
 
-		for i = 1,RAT7.MODES do
+		for i=1,RAT7.MODES do
 			self.Data[i] = {}
-			for j = 1, RAT7.BUTTONS do
-				self.Data[i][j] = {};
+			for j=1,RAT7.BUTTONS do
+				self.Data[i][j] = {}
 			end
 		end
 
 		self.InitSaveData = function(data)
-			for i = 1,RAT7.MODES do
-				if (nil == data["Rat"][i]) then
+			for i=1,RAT7.MODES do
+				if not data["Rat"][i] then
 					data["Rat"][i] = {}
 				end
-				for j = 1, RAT7.BUTTONS do
-					if (nil == data["Rat"][i][j]) then
+				for j=1,RAT7.BUTTONS do
+					if not data["Rat"][i][j] then
 						data["Rat"][i][j] = {}
 					end
 				end
 			end
 		end
 
-
 		self.LoadData = function()
 			msg("Loading...")
-			local data = CyborgMMO_GetSaveData();
+			local data = CyborgMMO_GetSaveData()
 
-
-			if (nil == data["Rat"]) then
+			if not data["Rat"] then
 				data["Rat"] = {}
-				self.InitSaveData(data);
+				self.InitSaveData(data)
 			end
 
 			self.Data = data["Rat"]
-			if(data ~= nil) then
-				for mode = 1,RAT7.MODES do
-					for button = 1, RAT7.BUTTONS do
-						if(self.Data[mode][button] ~= nil) then
-							object = CyborgMMO_WowObject.Create(self.Data[mode][button].Type, self.Data[mode][button].Detail, self.Data[mode][button].Subdetail);
-							self.SetObjectOnButtonNoUpdate(button, mode, object);
+			if data then
+				for mode=1,RAT7.MODES do
+					for button=1,RAT7.BUTTONS do
+						if self.Data[mode][button] then
+							object = CyborgMMO_WowObject.Create(self.Data[mode][button].Type, self.Data[mode][button].Detail, self.Data[mode][button].Subdetail)
+							self.SetObjectOnButtonNoUpdate(button, mode, object)
 						else
-							object = CyborgMMO_WowObject.Create("", "", "");
-							self.SetObjectOnButtonNoUpdate(button, mode, object);
-							self.Data[mode][button] = object;
+							object = CyborgMMO_WowObject.Create("", "", "")
+							self.SetObjectOnButtonNoUpdate(button, mode, object)
+							self.Data[mode][button] = object
 						end
 					end
 				end
-				self.UpdateObservers();
+				self.UpdateObservers()
 			end
 		end
 
 		self.SaveData = function()
 			msg("Saving...")
-			CyborgMMO_SetSaveData(self.Data, "Rat");
+			CyborgMMO_SetSaveData(self.Data, "Rat")
 		end
 
 		self.SetMode = function(mode)
-			self.m_Mode = mode;
-			self.UpdateObservers();
+			self.m_Mode = mode
+			self.UpdateObservers()
 		end
 
 		self.GetMode = function()
-			return self.m_Mode;
+			return self.m_Mode
 		end
 
 		self.GetData = function()
-			return self.Data, self.m_Mode;
+			return self.Data,self.m_Mode
 		end
 
 		self.GetObjectOnButton = function(button)
-			if(nil == self.Data[self.m_Mode][button]) then
-				return nil;
+			if not self.Data[self.m_Mode][button] then
+				return nil
 			else
 				return self.Data[self.m_Mode][button]
 			end
 		end
 
 		self.SetObjectOnButtonNoUpdate = function(button, mode, object)
-			--msg("button = "..tostring(button).." mode = "..tostring(mode))
-			self.Data[mode][button] = object;
+		--	msg("button = "..tostring(button).." mode = "..tostring(mode))
+			self.Data[mode][button] = object
 
-			if(nil ~= object) then
-				object.SetBinding(CyborgMMO_WowCommands[((mode-1)*RAT7.BUTTONS)+button]);
+			if object then
+				object.SetBinding(CyborgMMO_WowCommands[((mode-1)*RAT7.BUTTONS)+button])
 				if("callback" == object.Type) then
 					msg("trying to set texture")
-					local slot = getglobal("CyborgMMO_MainPageSlotListSlot"..button);
+					local slot = getglobal("CyborgMMO_MainPageSlotListSlot"..button)
 					slot:SetNormalTexture(object.Texture)
 				end
 			else
@@ -124,34 +126,36 @@
 		end
 
 		self.SetObjectOnButton = function(button, mode, object)
-			self.SetObjectOnButtonNoUpdate(button, mode, object);
+			self.SetObjectOnButtonNoUpdate(button, mode, object)
 			self.UpdateObservers()
 		end
 
 		self.AddObserver = function(view)
-			table.insert(self.ObserverList,view)
-			self.observerCount = # self.ObserverList
+			table.insert(self.ObserverList, view)
+			self.observerCount = #self.ObserverList
 		end
-		
+
 		self.GetAllObservers = function()
-			return self.ObserverList;
+			return self.ObserverList
 		end
 
 		self.UpdateObservers = function()
-			for i = 1, (# self.ObserverList) do
+			for i=1,#self.ObserverList do
 				self.ObserverList[i].Update(self.Data, self.m_Mode)
 			end
 			self.SaveData()
 		end
-	return self;
+
+		return self
 	end,
 
 	m_Instance = nil,
 
 	Instance = function()
-		if(nil == CyborgMMO_RatPageModel.m_Instance) then
-			CyborgMMO_RatPageModel.m_Instance = CyborgMMO_RatPageModel.new();
+		if not CyborgMMO_RatPageModel.m_Instance then
+			CyborgMMO_RatPageModel.m_Instance = CyborgMMO_RatPageModel.new()
 		end
-		return CyborgMMO_RatPageModel.m_Instance;
-	end
+		return CyborgMMO_RatPageModel.m_Instance
+	end,
 }
+