[Scummvm-cvs-logs] SF.net SVN: scummvm: [23961] scummvm/branches/branch-0-9-0

[sev at users.sourceforge.net]

Revision: 23961
          http://svn.sourceforge.net/scummvm/?rev=23961&view=rev
Author:   sev
Date:     2006-09-21 04:51:06 -0700 (Thu, 21 Sep 2006)

Log Message:
-----------
Backport fix for bug #1562795: "Bug with auto-generated game IDs"

Modified Paths:
--------------
    scummvm/branches/branch-0-9-0/NEWS
    scummvm/branches/branch-0-9-0/gui/launcher.cpp

Modified: scummvm/branches/branch-0-9-0/NEWS
===================================================================
--- scummvm/branches/branch-0-9-0/NEWS	2006-09-21 11:42:44 UTC (rev 23960)
+++ scummvm/branches/branch-0-9-0/NEWS	2006-09-21 11:51:06 UTC (rev 23961)
@@ -4,6 +4,7 @@
 0.9.1 (????-??-??)
  General:
    - Fixed potential garbage and/or crash in debug console.
+   - Removed restriction on 27 games per game id when added via launcher.
 
  SCUMM:
    - Improved support for international versions of HE games.

Modified: scummvm/branches/branch-0-9-0/gui/launcher.cpp
===================================================================
--- scummvm/branches/branch-0-9-0/gui/launcher.cpp	2006-09-21 11:42:44 UTC (rev 23960)
+++ scummvm/branches/branch-0-9-0/gui/launcher.cpp	2006-09-21 11:51:06 UTC (rev 23961)
@@ -642,13 +642,13 @@
 			// are *adding* a game to the config, not replacing).
 			String domain(result.gameid);
 			if (ConfMan.hasGameDomain(domain)) {
-				char suffix = 'a';
-				domain += suffix;
+				int suffixN = 1;
+				char suffix[16];
+
 				while (ConfMan.hasGameDomain(domain)) {
-					assert(suffix < 'z');
-					domain.deleteLastChar();
-					suffix++;
-					domain += suffix;
+					snprintf(suffix, 16, "-%d", suffixN);
+					domain = result.gameid + suffix;
+					suffixN++;
 				}
 			}
 


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