From 1f18d415091160ae9a8dd9594f22ba6991eab31f Mon Sep 17 00:00:00 2001
From: Don Honerbrink <dhonerbrink@gmail.com>
Date: Mon, 3 Aug 2015 09:35:14 -0500
Subject: [PATCH] Multiple joystick support. Createcollections argument creates
 menu.txt instead of menu.xml

---
 RetroFE/Source/Collection/CollectionInfoBuilder.cpp | 13 +------------
 RetroFE/Source/Control/UserInput.cpp                |  8 +++++++-
 RetroFE/Source/Control/UserInput.h                  |  5 +++--
 3 files changed, 11 insertions(+), 15 deletions(-)

diff --git a/RetroFE/Source/Collection/CollectionInfoBuilder.cpp b/RetroFE/Source/Collection/CollectionInfoBuilder.cpp
index e5131de..e7925ec 100644
--- a/RetroFE/Source/Collection/CollectionInfoBuilder.cpp
+++ b/RetroFE/Source/Collection/CollectionInfoBuilder.cpp
@@ -133,21 +133,10 @@ bool CollectionInfoBuilder::createCollectionDirectory(std::string name)
     settingsFile << "#media.video         = " << Utils::combinePath("%BASE_MEDIA_PATH%", "%ITEM_COLLECTION_NAME%", "medium_artwork", "video") << std::endl;
     settingsFile.close();
 
-    filename = Utils::combinePath(collectionPath, "menu.xml");
+    filename = Utils::combinePath(collectionPath, "menu.txt");
     std::cout << "Creating file \"" << filename << "\"" << std::endl;
     std::ofstream menuFile;
     menuFile.open(filename.c_str());
-
-    menuFile << "<menu>" << std::endl;
-    menuFile << std::endl;
-    menuFile << "<!-- uncomment this line and edit the example below to have a submenu" << std::endl;
-    menuFile << std::endl;
-    menuFile << "    <item collection=\"Some collection name\"/>" << std::endl;
-    menuFile << "    <item collection=\"Arcade\"/>" << std::endl;
-    menuFile << std::endl;
-    menuFile << "uncomment this line and edit the example above to have a submenu -->" << std::endl;
-    menuFile << std::endl;
-    menuFile << "</menu>" << std::endl;
     menuFile.close();
 
     return true;
diff --git a/RetroFE/Source/Control/UserInput.cpp b/RetroFE/Source/Control/UserInput.cpp
index 8b3e62a..b8f3d75 100644
--- a/RetroFE/Source/Control/UserInput.cpp
+++ b/RetroFE/Source/Control/UserInput.cpp
@@ -26,7 +26,6 @@
 
 UserInput::UserInput(Configuration &c)
     : config_(c)
-    , joystick_(NULL)
 {
     for(unsigned int i = 0; i < KeyCodeMax; ++i)
     {
@@ -37,6 +36,13 @@ UserInput::UserInput(Configuration &c)
 
 UserInput::~UserInput()
 {
+    for(std::vector<SDL_Joystick *>::iterator it = joysticks_.begin(); it != joysticks_.end(); it++)
+    {
+        if(*it)
+        {
+            SDL_JoystickClose(*it);
+        }
+    }
 }
 
 bool UserInput::initialize()
diff --git a/RetroFE/Source/Control/UserInput.h b/RetroFE/Source/Control/UserInput.h
index 4fbb66a..610c995 100644
--- a/RetroFE/Source/Control/UserInput.h
+++ b/RetroFE/Source/Control/UserInput.h
@@ -14,10 +14,11 @@
  * along with RetroFE.  If not, see <http://www.gnu.org/licenses/>.
  */
 #pragma once
-#include <map>
 #include <SDL2/SDL.h>
 #include <SDL2/SDL_joystick.h>
+#include <map>
 #include <string>
+#include <vector>
 
 class Configuration;
 class InputHandler;
@@ -54,7 +55,7 @@ public:
 private:
     bool MapKey(std::string keyDescription, KeyCode_E key);
     Configuration &config_;
-    SDL_Joystick *joystick_;
+    std::vector<SDL_Joystick *> joysticks_;
     InputHandler *keyHandlers_[KeyCodeMax]; 
     bool lastKeyState_[KeyCodeMax];