PennMUSH Community

Changeset 1061

Show
Ignore:
Timestamp:
08/05/07 01:55:26 (1 year ago)
Author:
shawnw
Message:

Merge db save failure fixes from devel into trunk

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • 1.8.2/trunk/CHANGES.182

    r1027 r1061  
    1313 
    1414========================================================================== 
     15 
     16Version 1.8.2 patchlevel 7                       ????  ??, 2007 
     17 
     18Fixes: 
     19  * Clarified the behavior of eval() and get_eval() in help. 
     20    Suggested by Talvo and Javelin. 
     21  * A failed db save no longer broadcasts a success message 
     22    in addition to a failure one. Reported by Cooee. 
     23  * The open database file wasn't getting closed on a failed 
     24    save. 
    1525 
    1626Version 1.8.2 patchlevel 6                       July 9, 2007 
  • 1.8.2/trunk/src/game.c

    r1027 r1061  
    339339    flag_broadcast("WIZARD ROYALTY", 0, 
    340340                   T("GAME: ERROR! Database save failed!")); 
     341    if (f) 
     342      db_close(f); 
    341343#ifndef PROFILING 
    342344#ifdef HAS_ITIMER 
     
    522524 
    523525  do_rawlog(LT_ERR, "DUMPING: %s.#%d#", globals.dumpfile, epoch); 
    524   dump_database_internal(); 
    525   do_rawlog(LT_ERR, "DUMPING: %s.#%d# (done)", globals.dumpfile, epoch); 
     526  if (!dump_database_internal()) 
     527    do_rawlog(LT_ERR, "DUMPING: %s.#%d# (done)", globals.dumpfile, epoch); 
    526528} 
    527529 
     
    616618    } else { 
    617619      reserve_fd(); 
    618       if (DUMP_NOFORK_COMPLETE && *DUMP_NOFORK_COMPLETE) 
     620      if (!status && DUMP_NOFORK_COMPLETE && *DUMP_NOFORK_COMPLETE) 
    619621        flag_broadcast(0, 0, "%s", DUMP_NOFORK_COMPLETE); 
    620622    }