[Scummvm-cvs-logs] SF.net SVN: scummvm:[35975] scummvm/trunk/backends/keymapper

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Wed Jan 21 03:28:14 CET 2009


Revision: 35975
          http://scummvm.svn.sourceforge.net/scummvm/?rev=35975&view=rev
Author:   fingolfin
Date:     2009-01-21 02:28:14 +0000 (Wed, 21 Jan 2009)

Log Message:
-----------
Yet more cleanup

Modified Paths:
--------------
    scummvm/trunk/backends/keymapper/keymapper.h
    scummvm/trunk/backends/keymapper/remap-dialog.cpp
    scummvm/trunk/backends/keymapper/remap-dialog.h

Modified: scummvm/trunk/backends/keymapper/keymapper.h
===================================================================
--- scummvm/trunk/backends/keymapper/keymapper.h	2009-01-21 02:23:09 UTC (rev 35974)
+++ scummvm/trunk/backends/keymapper/keymapper.h	2009-01-21 02:28:14 UTC (rev 35975)
@@ -168,7 +168,7 @@
 
 	Domain& getGlobalDomain() { return _globalDomain; }
 	Domain& getGameDomain() { return _gameDomain; }
-	Stack<MapRecord>& getActiveStack() { return _activeMaps; }
+	const Stack<MapRecord>& getActiveStack() const { return _activeMaps; }
 
 private:
 

Modified: scummvm/trunk/backends/keymapper/remap-dialog.cpp
===================================================================
--- scummvm/trunk/backends/keymapper/remap-dialog.cpp	2009-01-21 02:23:09 UTC (rev 35974)
+++ scummvm/trunk/backends/keymapper/remap-dialog.cpp	2009-01-21 02:28:14 UTC (rev 35975)
@@ -57,9 +57,9 @@
 
 void RemapDialog::open() {
 	bool divider = false;
-	_activeKeymaps = &_keymapper->getActiveStack();
-	if (_activeKeymaps->size() > 0) {
-		_kmPopUp->appendEntry(_activeKeymaps->top().keymap->getName() + " (Active)");
+	const Stack<Keymapper::MapRecord> &activeKeymaps = _keymapper->getActiveStack();
+	if (!activeKeymaps.size() > 0) {
+		_kmPopUp->appendEntry(activeKeymaps.top().keymap->getName() + " (Active)");
 		divider = true;
 	}
 
@@ -232,13 +232,14 @@
 
 void RemapDialog::loadKeymap() {
 	_currentActions.clear();
-	if (_activeKeymaps->size() > 0 && _kmPopUp->getSelected() == 0) {
+	const Stack<Keymapper::MapRecord> &activeKeymaps = _keymapper->getActiveStack();
+	if (!activeKeymaps.empty() && _kmPopUp->getSelected() == 0) {
 		// load active keymaps
 
-		List<const HardwareKey*> freeKeys (_keymapper->getHardwareKeys());
+		List<const HardwareKey*> freeKeys(_keymapper->getHardwareKeys());
 
 		// add most active keymap's keys
-		Keymapper::MapRecord top = _activeKeymaps->top();
+		Keymapper::MapRecord top = activeKeymaps.top();
 		List<Action*>::iterator actIt;
 		for (actIt = top.keymap->getActions().begin(); actIt != top.keymap->getActions().end(); ++actIt) {
 			Action *act = *actIt;
@@ -250,8 +251,8 @@
 		
 		// loop through remaining finding mappings for unmapped keys
 		if (top.inherit) {
-			for (int i = _activeKeymaps->size() - 2; i >= 0; --i) {
-				Keymapper::MapRecord mr = (*_activeKeymaps)[i];
+			for (int i = activeKeymaps.size() - 2; i >= 0; --i) {
+				Keymapper::MapRecord mr = activeKeymaps[i];
 				List<const HardwareKey*>::iterator keyIt = freeKeys.begin();
 				while (keyIt != freeKeys.end()) {
 					Action *act = mr.keymap->getMappedAction((*keyIt)->key);
@@ -288,7 +289,8 @@
 
 void RemapDialog::refreshKeymap() {
 	int newTopAction = _scrollBar->_currentPos * _colCount;
-	if (newTopAction == _topAction) return;
+	if (newTopAction == _topAction)
+		return;
 	_topAction = newTopAction;
 
 	//_container->draw();
@@ -306,12 +308,12 @@
 				widg.keyButton->setLabel(mappedKey->description);
 			else
 				widg.keyButton->setLabel("-");
-			widg.actionText->clearFlags(GUI::WIDGET_INVISIBLE);
-			widg.keyButton->clearFlags(GUI::WIDGET_INVISIBLE);
+			widg.actionText->setVisible(true);
+			widg.keyButton->setVisible(true);
 			actionI++; 
 		} else {
-			widg.actionText->setFlags(GUI::WIDGET_INVISIBLE);
-			widg.keyButton->setFlags(GUI::WIDGET_INVISIBLE);
+			widg.actionText->setVisible(false);
+			widg.keyButton->setVisible(false);
 		}
 		//widg.actionText->draw();
 		//widg.keyButton->draw();

Modified: scummvm/trunk/backends/keymapper/remap-dialog.h
===================================================================
--- scummvm/trunk/backends/keymapper/remap-dialog.h	2009-01-21 02:23:09 UTC (rev 35974)
+++ scummvm/trunk/backends/keymapper/remap-dialog.h	2009-01-21 02:28:14 UTC (rev 35975)
@@ -68,7 +68,6 @@
 	void stopRemapping();
 
 	Keymapper *_keymapper;
-	Stack<Keymapper::MapRecord> *_activeKeymaps;
 	Keymap** _keymapTable;
 
 	Array<ActionInfo> _currentActions;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list