[InputMapper] Use "|" for separating entries instead of ",", since some companies have names like "x Ltd., something". [Mordae, AJ]

Docs/Changelog_sm5.txt

@@ -8,6 +8,11 @@ ________________________________________________________________________________
 StepMania 5.0 ???? ?? | 20120???
 --------------------------------------------------------------------------------
 
+2012/07/09
+----------
+* [InputMapper] Use "|" for separating entries instead of ",", since some
+  companies have names like "x Ltd., something". [Mordae, AJ]
+
 2012/07/06
 ----------
 * [ScoreDisplayRave] Add FrameBase and FrameOver to the metrics. [AJ]

src/InputMapper.cpp

@@ -667,7 +667,9 @@ bool InputMapper::CheckForChangedInputDevicesAndRemap( RString &sMessageOut )
 
 	// Strip non-joysticks.
 	vector<RString> vsLastSeenJoysticks;
-	split( g_sLastSeenInputDevices, ",", vsLastSeenJoysticks );
+	// Don't use "," since some vendors have a name like "company Ltd., etc".
+	// For now, use a pipe character. -aj, fix from Mordae.
+	split( g_sLastSeenInputDevices, "|", vsLastSeenJoysticks );
 
 	vector<RString> vsCurrent;
 	vector<RString> vsCurrentJoysticks;
@@ -709,7 +711,8 @@ bool InputMapper::CheckForChangedInputDevicesAndRemap( RString &sMessageOut )
 
 	LOG->Info( "%s", sMessageOut.c_str() );
 
-	g_sLastSeenInputDevices.Set( join(",",vsCurrent) );
+	// see above comment about not using ",". -aj
+	g_sLastSeenInputDevices.Set( join("|",vsCurrent) );
 	PREFSMAN->SavePrefsToDisk();
 
 	return true;