mail[Wesnoth-commits] r24289 - /trunk/src/playcampaign.cpp


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by esr on March 04, 2008 - 00:47:
Author: esr
Date: Tue Mar  4 00:31:17 2008
New Revision: 24289

URL: http://svn.gna.org/viewcvs/wesnoth?rev=24289&view=rev
Log:
Document a harmless bug with some architecture implications.

Modified:
    trunk/src/playcampaign.cpp

Modified: trunk/src/playcampaign.cpp
URL: 
http://svn.gna.org/viewcvs/wesnoth/trunk/src/playcampaign.cpp?rev=24289&r1=24288&r2=24289&view=diff
==============================================================================
--- trunk/src/playcampaign.cpp (original)
+++ trunk/src/playcampaign.cpp Tue Mar  4 00:31:17 2008
@@ -339,8 +339,16 @@
                                }
                        }
 
-                       // This is the magic moment when game history links
-                       // are created
+                       // This is the magic moment when game history IDs and
+                       // ancestry links are created.  Note: Doing it this
+                       // way has a harmless bug -- an extra ID will be 
+                       // generated here for start-of-campaign saves.  This
+                       // is really an artifact of the messiness of the 
+                       // save logic and should be fixed when that is.  There
+                       // is only one other call to generate_game_uuid(), and
+                       // that is at campaign initialization time; it is
+                       // needed to make sure 'history' is a top-level
+                       // attribute.
                        if (!gamestate.history.empty())
                                gamestate.history += ",";
                        gamestate.history += generate_game_uuid();




Related Messages


Powered by MHonArc, Updated Tue Mar 04 01:01:46 2008