From c7a9637834a96722be5a524c4c1e8932a45e3461 Mon Sep 17 00:00:00 2001 From: smatz Date: Tue, 3 Jun 2008 18:35:58 +0000 Subject: (svn r13375) -Add: logging of actions that could possibly cause desyncs and crashes to simplify debugging. See readme.txt for details --- src/gamelog.cpp | 801 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 801 insertions(+) create mode 100644 src/gamelog.cpp (limited to 'src/gamelog.cpp') diff --git a/src/gamelog.cpp b/src/gamelog.cpp new file mode 100644 index 000000000..e5518dbe8 --- /dev/null +++ b/src/gamelog.cpp @@ -0,0 +1,801 @@ +/* $Id$ */ + +/** @file gamelog.cpp Definition of functions used for logging of important changes in the game */ + +#include "stdafx.h" +#include "openttd.h" +#include "saveload.h" +#include "core/alloc_func.hpp" +#include "core/bitmath_func.hpp" +#include "core/math_func.hpp" +#include "network/core/config.h" +#include "variables.h" +#include "string_func.h" +#include "settings_type.h" +#include "newgrf_config.h" +#include +#include +#include "gamelog.h" +#include "console_func.h" +#include "debug.h" +#include "rev.h" + +extern const uint16 SAVEGAME_VERSION; ///< current savegame version + +extern SavegameType _savegame_type; ///< type of savegame we are loading + +extern uint32 _ttdp_version; ///< version of TTDP savegame (if applicable) +extern uint16 _sl_version; ///< the major savegame version identifier +extern byte _sl_minor_version; ///< the minor savegame version, DO NOT USE! + +/** Type of logged change */ +enum GamelogChangeType { + GLCT_MODE, ///< Scenario editor x Game, different landscape + GLCT_REVISION, ///< Changed game revision string + GLCT_OLDVER, ///< Loaded from savegame without logged data + GLCT_PATCH, ///< Non-networksafe patch value changed + GLCT_GRFADD, ///< Removed GRF + GLCT_GRFREM, ///< Added GRF + GLCT_GRFCOMPAT, ///< Loading compatible GRF + GLCT_GRFPARAM, ///< GRF parameter changed + GLCT_GRFMOVE, ///< GRF order changed + GLCT_END, ///< So we know how many GLCTs are there + GLCT_NONE = 0xFF, ///< In savegames, end of list +}; + + +/** Contains information about one logged change */ +struct LoggedChange { + GamelogChangeType ct; ///< Type of change logged in this struct + union { + struct { + byte mode; ///< new game mode - Editor x Game + byte landscape; ///< landscape (temperate, arctic, ...) + } mode; + struct { + char text[NETWORK_REVISION_LENGTH]; ///< revision string, _openttd_revision + uint32 newgrf; ///< _openttd_newgrf_version + uint16 slver; ///< _sl_version + byte modified; ///< _openttd_revision_modified + } revision; + struct { + uint32 type; ///< type of savegame, @see SavegameType + uint32 version; ///< major and minor version OR ttdp version + } oldver; + GRFIdentifier grfadd; ///< ID and md5sum of added GRF + struct { + uint32 grfid; ///< ID of removed GRF + } grfrem; + GRFIdentifier grfcompat; ///< ID and new md5sum of changed GRF + struct { + uint32 grfid; ///< ID of GRF with changed parameters + } grfparam; + struct { + uint32 grfid; ///< ID of moved GRF + int32 offset; ///< offset, positive = move down + } grfmove; + struct { + char *name; ///< name of the patch + int32 oldval; ///< old value + int32 newval; ///< new value + } patch; + }; +}; + + +/** Contains information about one logged action that caused at least one logged change */ +struct LoggedAction { + LoggedChange *change; ///< First logged change in this action + uint32 changes; ///< Number of changes in this action + GamelogActionType at; ///< Type of action + uint16 tick; ///< Tick when it happened +}; + +static GamelogActionType _gamelog_action_type = GLAT_NONE; ///< action to record if anything changes + +static LoggedAction *_gamelog_action = NULL; ///< first logged action +static uint _gamelog_actions = 0; ///< number of actions +static LoggedAction *_current_action = NULL; ///< current action we are logging, NULL when there is no action active + + +/** Stores information about new action, but doesn't allocate it + * Action is allocated only when there is at least one change + * @param at type of action + */ +void GamelogStartAction(GamelogActionType at) +{ + assert(_gamelog_action_type == GLAT_NONE); // do not allow starting new action without stopping the previous first + _gamelog_action_type = at; +} + +/** Stops logging of any changes + */ +void GamelogStopAction() +{ + assert(_gamelog_action_type != GLAT_NONE); // nobody should try to stop if there is no action in progress + + _current_action = NULL; + _gamelog_action_type = GLAT_NONE; + + if (_debug_gamelog_level > 4) GamelogPrintDebug(); +} + +/** Resets and frees all memory allocated - used before loading or starting a new game + */ +void GamelogReset() +{ + assert(_gamelog_action_type == GLAT_NONE); + + for (uint i = 0; i < _gamelog_actions; i++) { + const LoggedAction *la = &_gamelog_action[i]; + for (uint j = 0; j < la->changes; j++) { + const LoggedChange *lc = &la->change[j]; + if (lc->ct == GLCT_PATCH) free(lc->patch.name); + } + free(la->change); + } + + free(_gamelog_action); + + _gamelog_action = NULL; + _gamelog_actions = 0; + _current_action = NULL; +} + +enum { + GAMELOG_BUF_LEN = 1024 ///< length of buffer for one line of text +}; + +static int _dbgofs = 0; ///< offset in current output buffer + +static void AddDebugText(char *buf, const char *s, ...) +{ + if (GAMELOG_BUF_LEN <= _dbgofs) return; + + va_list va; + + va_start(va, s); + _dbgofs += vsnprintf(buf + _dbgofs, GAMELOG_BUF_LEN - _dbgofs, s, va); + va_end(va); +} + + +/** Prints GRF filename if found + * @param grfid GRF which filename to print + */ +static void PrintGrfFilename(char *buf, uint grfid) +{ + const GRFConfig *gc = FindGRFConfig(grfid); + + if (gc == NULL) return; + + AddDebugText(buf, ", filename: %s", gc->filename); +} + +/** Prints GRF ID, checksum and filename if found + * @param grfid GRF ID + * @param md5sum array of md5sum to print + */ +static void PrintGrfInfo(char *buf, uint grfid, const uint8 *md5sum) +{ + char txt[40]; + + md5sumToString(txt, lastof(txt), md5sum); + + AddDebugText(buf, "GRF ID %08X, checksum %s", BSWAP32(grfid), txt); + + PrintGrfFilename(buf, grfid); + + return; +} + + +/** Text messages for various logged actions */ +static const char *la_text[] = { + "new game started", + "game loaded", + "GRF config changed", + "cheat was used", + "patch settings changed" +}; + +assert_compile(lengthof(la_text) == GLAT_END); + + +/** Prints active gamelog */ +void GamelogPrint(GamelogPrintProc *proc) +{ + char buf[GAMELOG_BUF_LEN]; + + proc("---- gamelog start ----"); + + const LoggedAction *laend = &_gamelog_action[_gamelog_actions]; + + for (const LoggedAction *la = _gamelog_action; la != laend; la++) { + assert((uint)la->at < GLAT_END); + + snprintf(buf, GAMELOG_BUF_LEN, "Tick %u: %s", (uint)la->tick, la_text[(uint)la->at]); + proc(buf); + + const LoggedChange *lcend = &la->change[la->changes]; + + for (const LoggedChange *lc = la->change; lc != lcend; lc++) { + _dbgofs = 0; + AddDebugText(buf, " "); + + switch (lc->ct) { + default: NOT_REACHED(); + case GLCT_MODE: + AddDebugText(buf, "New game mode: %u landscape: %u", + (uint)lc->mode.mode, (uint)lc->mode.landscape); + break; + + case GLCT_REVISION: + AddDebugText(buf, "Revision text changed to %s, savegame version %u, ", + lc->revision.text, lc->revision.slver); + + switch (lc->revision.modified) { + case 0: AddDebugText(buf, "not "); break; + case 1: AddDebugText(buf, "maybe "); break; + default: break; + } + + AddDebugText(buf, "modified, _openttd_newgrf_version = 0x%08x", lc->revision.newgrf); + break; + + case GLCT_OLDVER: + AddDebugText(buf, "Conversion from "); + switch (lc->oldver.type) { + default: NOT_REACHED(); + case SGT_OTTD: + AddDebugText(buf, "OTTD savegame without gamelog: version %u, %u", + GB(lc->oldver.version, 8, 16), GB(lc->oldver.version, 0, 8)); + break; + + case SGT_TTD: + AddDebugText(buf, "TTD savegame"); + break; + + case SGT_TTDP1: + case SGT_TTDP2: + AddDebugText(buf, "TTDP savegame, %s format", + lc->oldver.type == SGT_TTDP1 ? "old" : "new"); + if (lc->oldver.version != 0) { + AddDebugText(buf, ", TTDP version %u.%u.%u.%u", + GB(lc->oldver.version, 24, 8), GB(lc->oldver.version, 20, 4), + GB(lc->oldver.version, 16, 4), GB(lc->oldver.version, 0, 16)); + } + break; + } + break; + + case GLCT_PATCH: + AddDebugText(buf, "Patch setting changed: %s : %d -> %d", lc->patch.name, lc->patch.oldval, lc->patch.newval); + break; + + case GLCT_GRFADD: + AddDebugText(buf, "Added NewGRF: "); + PrintGrfInfo(buf, lc->grfadd.grfid, lc->grfadd.md5sum); + break; + + case GLCT_GRFREM: + AddDebugText(buf, "Removed NewGRF: %08X", BSWAP32(lc->grfrem.grfid)); + PrintGrfFilename(buf, lc->grfrem.grfid); + break; + + case GLCT_GRFCOMPAT: + AddDebugText(buf, "Compatible NewGRF loaded: "); + PrintGrfInfo(buf, lc->grfcompat.grfid, lc->grfcompat.md5sum); + break; + + case GLCT_GRFPARAM: + AddDebugText(buf, "GRF parameter changed: %08X", BSWAP32(lc->grfparam.grfid)); + PrintGrfFilename(buf, lc->grfparam.grfid); + break; + + case GLCT_GRFMOVE: + AddDebugText(buf, "GRF order changed: %08X moved %d places %s", + BSWAP32(lc->grfmove.grfid), abs(lc->grfmove.offset), lc->grfmove.offset >= 0 ? "down" : "up" ); + PrintGrfFilename(buf, lc->grfmove.grfid); + break; + } + + proc(buf); + } + } + + proc("---- gamelog end ----"); +} + + +static void GamelogPrintConsoleProc(const char *s) +{ + IConsolePrint(CC_WARNING, s); +} + +void GamelogPrintConsole() +{ + GamelogPrint(&GamelogPrintConsoleProc); +} + + +static void GamelogPrintDebugProc(const char *s) +{ + debug_print("gamelog", s); +} + +void GamelogPrintDebug() +{ + GamelogPrint(&GamelogPrintDebugProc); +} + + +/** Allocates new LoggedChange and new LoggedAction if needed. + * If there is no action active, NULL is returned. + * @param ct type of change + * @return new LoggedChange, or NULL if there is no action active + */ +static LoggedChange *GamelogChange(GamelogChangeType ct) +{ + if (_current_action == NULL) { + if (_gamelog_action_type == GLAT_NONE) return NULL; + + _gamelog_action = ReallocT(_gamelog_action, _gamelog_actions + 1); + _current_action = &_gamelog_action[_gamelog_actions++]; + + _current_action->at = _gamelog_action_type; + _current_action->tick = _tick_counter; + _current_action->change = NULL; + _current_action->changes = 0; + } + + _current_action->change = ReallocT(_current_action->change, _current_action->changes + 1); + + LoggedChange *lc = &_current_action->change[_current_action->changes++]; + lc->ct = ct; + + return lc; +} + + +/** Logs a change in game revision + * @param revision new revision string + */ +void GamelogRevision() +{ + assert(_gamelog_action_type == GLAT_START || _gamelog_action_type == GLAT_LOAD); + + LoggedChange *lc = GamelogChange(GLCT_REVISION); + if (lc == NULL) return; + + strncpy(lc->revision.text, _openttd_revision, lengthof(lc->revision.text)); + lc->revision.slver = SAVEGAME_VERSION; + lc->revision.modified = _openttd_revision_modified; + lc->revision.newgrf = _openttd_newgrf_version; +} + +/** Logs a change in game mode (scenario editor or game) + */ +void GamelogMode() +{ + assert(_gamelog_action_type == GLAT_START || _gamelog_action_type == GLAT_LOAD || _gamelog_action_type == GLAT_CHEAT); + + LoggedChange *lc = GamelogChange(GLCT_MODE); + if (lc == NULL) return; + + lc->mode.mode = _game_mode; + lc->mode.landscape = _settings_game.game_creation.landscape; +} + +/** Logs loading from savegame without gamelog + */ +void GamelogOldver() +{ + assert(_gamelog_action_type == GLAT_LOAD); + + LoggedChange *lc = GamelogChange(GLCT_OLDVER); + if (lc == NULL) return; + + lc->oldver.type = _savegame_type; + lc->oldver.version = (_savegame_type == SGT_OTTD ? ((uint32)_sl_version << 8 | _sl_minor_version) : _ttdp_version); +} + +/** Logs change in game patches. Only non-networksafe patches are logged + * @param name patch name + * @param oldval old patch value + * @param newval new patch value + */ +void GamelogPatch(const char *name, int32 oldval, int32 newval) +{ + assert(_gamelog_action_type == GLAT_PATCH); + + LoggedChange *lc = GamelogChange(GLCT_PATCH); + if (lc == NULL) return; + + lc->patch.name = strdup(name); + lc->patch.oldval = oldval; + lc->patch.newval = newval; +} + + +/** Finds out if current revision is different than last revision stored in the savegame. + * Appends GLCT_REVISION when the revision string changed + */ +void GamelogTestRevision() +{ + const LoggedChange *rev = NULL; + + const LoggedAction *laend = &_gamelog_action[_gamelog_actions]; + for (const LoggedAction *la = _gamelog_action; la != laend; la++) { + const LoggedChange *lcend = &la->change[la->changes]; + for (const LoggedChange *lc = la->change; lc != lcend; lc++) { + if (lc->ct == GLCT_REVISION) rev = lc; + } + } + + if (rev == NULL || strcmp(rev->revision.text, _openttd_revision) != 0 || + rev->revision.modified != _openttd_revision_modified || + rev->revision.newgrf != _openttd_newgrf_version) { + GamelogRevision(); + } +} + +/** Finds last stored game mode or landscape. + * Any change is logged + */ +void GamelogTestMode() +{ + const LoggedChange *mode = NULL; + + const LoggedAction *laend = &_gamelog_action[_gamelog_actions]; + for (const LoggedAction *la = _gamelog_action; la != laend; la++) { + const LoggedChange *lcend = &la->change[la->changes]; + for (const LoggedChange *lc = la->change; lc != lcend; lc++) { + if (lc->ct == GLCT_MODE) mode = lc; + } + } + + if (mode == NULL || mode->mode.mode != _game_mode || mode->mode.landscape != _settings_game.game_creation.landscape) GamelogMode(); +} + + +/** Decides if GRF should be logged + * @param g grf to determine + * @return true iff GRF is not static and is loaded + */ +static inline bool IsLoggableGrfConfig(const GRFConfig *g) +{ + return !HasBit(g->flags, GCF_STATIC) && g->status != GCS_NOT_FOUND; +} + +/** Logs removal of a GRF + * @param grfid ID of removed GRF + */ +void GamelogGRFRemove(uint32 grfid) +{ + assert(_gamelog_action_type == GLAT_LOAD || _gamelog_action_type == GLAT_GRF); + + LoggedChange *lc = GamelogChange(GLCT_GRFREM); + if (lc == NULL) return; + + lc->grfrem.grfid = grfid; +} + +/** Logs adding of a GRF + * @param newg added GRF + */ +void GamelogGRFAdd(const GRFConfig *newg) +{ + assert(_gamelog_action_type == GLAT_LOAD || _gamelog_action_type == GLAT_START || _gamelog_action_type == GLAT_GRF); + + if (!IsLoggableGrfConfig(newg)) return; + + LoggedChange *lc = GamelogChange(GLCT_GRFADD); + if (lc == NULL) return; + + memcpy(&lc->grfadd, newg, sizeof(GRFIdentifier)); +} + +/** Logs loading compatible GRF + * (the same ID, but different MD5 hash) + * @param newg new (updated) GRF + */ +void GamelogGRFCompatible(const GRFIdentifier *newg) +{ + assert(_gamelog_action_type == GLAT_LOAD); + + LoggedChange *lc = GamelogChange(GLCT_GRFCOMPAT); + if (lc == NULL) return; + + memcpy(&lc->grfcompat, newg, sizeof(GRFIdentifier)); +} + +/** Logs changing GRF order + * @param grfid GRF that is moved + * @param offset how far it is moved, positive = moved down + */ +static void GamelogGRFMove(uint32 grfid, int32 offset) +{ + assert(_gamelog_action_type == GLAT_GRF); + + LoggedChange *lc = GamelogChange(GLCT_GRFMOVE); + if (lc == NULL) return; + + lc->grfmove.grfid = grfid; + lc->grfmove.offset = offset; +} + +/** Logs change in GRF parameters. + * Details about parameters changed are not stored + * @param grfid ID of GRF to store + */ +static void GamelogGRFParameters(uint32 grfid) +{ + assert(_gamelog_action_type == GLAT_GRF); + + LoggedChange *lc = GamelogChange(GLCT_GRFPARAM); + if (lc == NULL) return; + + lc->grfparam.grfid = grfid; +} + +/** Logs adding of list of GRFs. + * Useful when old savegame is loaded or when new game is started + * @param newg head of GRF linked list + */ +void GamelogGRFAddList(const GRFConfig *newg) +{ + assert(_gamelog_action_type == GLAT_START || _gamelog_action_type == GLAT_LOAD); + + for (; newg != NULL; newg = newg->next) { + GamelogGRFAdd(newg); + } +} + +/** List of GRFs using array of pointers instead of linked list */ +struct GRFList { + uint n; + const GRFConfig *grf[VARARRAY_SIZE]; +}; + +/** Generates GRFList + * @param grfc head of GRF linked list + */ +static GRFList *GenerateGRFList(const GRFConfig *grfc) +{ + uint n = 0; + for (const GRFConfig *g = grfc; g != NULL; g = g->next) { + if (IsLoggableGrfConfig(g)) n++; + } + + GRFList *list = (GRFList*)MallocT(sizeof(GRFList) + n * sizeof(GRFConfig*)); + + list->n = 0; + for (const GRFConfig *g = grfc; g != NULL; g = g->next) { + if (IsLoggableGrfConfig(g)) list->grf[list->n++] = g; + } + + return list; +} + +/** Compares two NewGRF lists and logs any change + * @param oldc original GRF list + * @param newc new GRF list + */ +void GamelogGRFUpdate(const GRFConfig *oldc, const GRFConfig *newc) +{ + GRFList *ol = GenerateGRFList(oldc); + GRFList *nl = GenerateGRFList(newc); + + uint o = 0, n = 0; + + while (o < ol->n && n < nl->n) { + const GRFConfig *og = ol->grf[o]; + const GRFConfig *ng = nl->grf[n]; + + if (og->grfid != ng->grfid) { + uint oi, ni; + for (oi = 0; oi < ol->n; oi++) { + if (ol->grf[oi]->grfid == nl->grf[n]->grfid) break; + } + if (oi < o) { + /* GRF was moved, this change has been logged already */ + n++; + continue; + } + if (oi == ol->n) { + /* GRF couldn't be found in the OLD list, GRF was ADDED */ + GamelogGRFAdd(nl->grf[n++]); + continue; + } + for (ni = 0; ni < nl->n; ni++) { + if (nl->grf[ni]->grfid == ol->grf[o]->grfid) break; + } + if (ni < n) { + /* GRF was moved, this change has been logged already */ + o++; + continue; + } + if (ni == nl->n) { + /* GRF couldn't be found in the NEW list, GRF was REMOVED */ + GamelogGRFRemove(ol->grf[o++]->grfid); + continue; + } + + /* o < oi < ol->n + * n < ni < nl->n */ + assert(ni > n && ni < nl->n); + assert(oi > o && oi < ol->n); + + ni -= n; // number of GRFs it was moved downwards + oi -= o; // number of GRFs it was moved upwards + + if (ni >= oi) { // prefer the one that is moved further + /* GRF was moved down */ + GamelogGRFMove(ol->grf[o++]->grfid, ni); + } else { + GamelogGRFMove(nl->grf[n++]->grfid, -(int)oi); + } + } else { + if (memcmp(og->md5sum, ng->md5sum, sizeof(og->md5sum)) != 0) { + /* md5sum changed, probably loading 'compatible' GRF */ + GamelogGRFCompatible(nl->grf[n]); + } + + if (og->num_params != ng->num_params || memcmp(og->param, ng->param, og->num_params * sizeof(og->param[0])) != 0) { + GamelogGRFParameters(ol->grf[o]->grfid); + } + + o++; + n++; + } + } + + while (o < ol->n) GamelogGRFRemove(ol->grf[o++]->grfid); // remaining GRFs were removed ... + while (n < nl->n) GamelogGRFAdd (nl->grf[n++]); // ... or added + + free(ol); + free(nl); +} + + +static const SaveLoad _glog_action_desc[] = { + SLE_VAR(LoggedAction, tick, SLE_UINT16), + SLE_END() +}; + +static const SaveLoad _glog_mode_desc[] = { + SLE_VAR(LoggedChange, mode.mode, SLE_UINT8), + SLE_VAR(LoggedChange, mode.landscape, SLE_UINT8), + SLE_END() +}; + +static const SaveLoad _glog_revision_desc[] = { + SLE_ARR(LoggedChange, revision.text, SLE_UINT8, NETWORK_REVISION_LENGTH), + SLE_VAR(LoggedChange, revision.newgrf, SLE_UINT32), + SLE_VAR(LoggedChange, revision.slver, SLE_UINT16), + SLE_VAR(LoggedChange, revision.modified, SLE_UINT8), + SLE_END() +}; + +static const SaveLoad _glog_oldver_desc[] = { + SLE_VAR(LoggedChange, oldver.type, SLE_UINT32), + SLE_VAR(LoggedChange, oldver.version, SLE_UINT32), + SLE_END() +}; + +static const SaveLoad _glog_patch_desc[] = { + SLE_STR(LoggedChange, patch.name, SLE_STR, 128), + SLE_VAR(LoggedChange, patch.oldval, SLE_INT32), + SLE_VAR(LoggedChange, patch.newval, SLE_INT32), + SLE_END() +}; + +static const SaveLoad _glog_grfadd_desc[] = { + SLE_VAR(LoggedChange, grfadd.grfid, SLE_UINT32 ), + SLE_ARR(LoggedChange, grfadd.md5sum, SLE_UINT8, 16), + SLE_END() +}; + +static const SaveLoad _glog_grfrem_desc[] = { + SLE_VAR(LoggedChange, grfrem.grfid, SLE_UINT32), + SLE_END() +}; + +static const SaveLoad _glog_grfcompat_desc[] = { + SLE_VAR(LoggedChange, grfcompat.grfid, SLE_UINT32 ), + SLE_ARR(LoggedChange, grfcompat.md5sum, SLE_UINT8, 16), + SLE_END() +}; + +static const SaveLoad _glog_grfparam_desc[] = { + SLE_VAR(LoggedChange, grfparam.grfid, SLE_UINT32), + SLE_END() +}; + +static const SaveLoad _glog_grfmove_desc[] = { + SLE_VAR(LoggedChange, grfmove.grfid, SLE_UINT32), + SLE_VAR(LoggedChange, grfmove.offset, SLE_INT32), + SLE_END() +}; + +static const SaveLoad *_glog_desc[] = { + _glog_mode_desc, + _glog_revision_desc, + _glog_oldver_desc, + _glog_patch_desc, + _glog_grfadd_desc, + _glog_grfrem_desc, + _glog_grfcompat_desc, + _glog_grfparam_desc, + _glog_grfmove_desc +}; + +assert_compile(lengthof(_glog_desc) == GLCT_END); + +static void Load_GLOG() +{ + assert(_gamelog_action == NULL); + assert(_gamelog_actions == 0); + + GamelogActionType at; + while ((at = (GamelogActionType)SlReadByte()) != GLAT_NONE) { + _gamelog_action = ReallocT(_gamelog_action, _gamelog_actions + 1); + LoggedAction *la = &_gamelog_action[_gamelog_actions++]; + + la->at = at; + + SlObject(la, _glog_action_desc); // has to be saved after 'DATE'! + la->change = NULL; + la->changes = 0; + + GamelogChangeType ct; + while ((ct = (GamelogChangeType)SlReadByte()) != GLCT_NONE) { + la->change = ReallocT(la->change, la->changes + 1); + + LoggedChange *lc = &la->change[la->changes++]; + lc->ct = ct; + + assert((uint)ct < GLCT_END); + + SlObject(lc, _glog_desc[ct]); + } + } +} + +static void Save_GLOG() +{ + const LoggedAction *laend = &_gamelog_action[_gamelog_actions]; + size_t length = 0; + + for (const LoggedAction *la = _gamelog_action; la != laend; la++) { + const LoggedChange *lcend = &la->change[la->changes]; + for (LoggedChange *lc = la->change; lc != lcend; lc++) { + assert((uint)lc->ct < lengthof(_glog_desc)); + length += SlCalcObjLength(lc, _glog_desc[lc->ct]) + 1; + } + length += 4; + } + length++; + + SlSetLength(length); + + for (LoggedAction *la = _gamelog_action; la != laend; la++) { + SlWriteByte(la->at); + SlObject(la, _glog_action_desc); + + const LoggedChange *lcend = &la->change[la->changes]; + for (LoggedChange *lc = la->change; lc != lcend; lc++) { + SlWriteByte(lc->ct); + assert((uint)lc->ct < GLCT_END); + SlObject(lc, _glog_desc[lc->ct]); + } + SlWriteByte(GLCT_NONE); + } + SlWriteByte(GLAT_NONE); +} + + +extern const ChunkHandler _gamelog_chunk_handlers[] = { + { 'GLOG', Save_GLOG, Load_GLOG, CH_RIFF | CH_LAST } +}; -- cgit v1.2.3-54-g00ecf