timings for opening and closing sector inventory with 251 items at D13 when Booby Rays shipment has arrived but not yet oppened.
because of rondomness on loading times I repeated it 5 times for each revision I tested.
After each test I reloaded the save.
I accessed inventory via left Btn mouseclick and closed via [ESC].
rev open1 close1 open2 close2 open3 close3 open4 close4 open5 close5

6401 11248 6911 9360 7285 10421 7332 12527 7878 12293 7411
6387 11559 7394 12839 7067 11435 7130 9282 7800 10842 7129
6382 9017 6724 10749 7254 10499 7347 10655 7348 12074 6958
6375 9017 6974 10030 7301 11498 7520 11607 7269 13042 6958
6370 11248 7020 11279 7332 10671 7192 9843 7394 10640 7379
6367 10499 7222 9001 7160 10920 7332 9438 7191 12948 7145

Index: gameloop.cpp
===================================================================
--- gameloop.cpp (revision 6401)
+++ gameloop.cpp (working copy)
@@ -220,7 +220,7 @@

// rain
static BOOLEAN gfSkipFrame = FALSE;
-
+extern void ScreenMsg( UINT16 usColor, UINT8 ubPriority, STR16 pStringA, ...);
void GameLoop(void)
{
// DebugMsg (TOPIC_JA2,DBG_LEVEL_3,"GameLoop");
@@ -377,8 +377,11 @@

//DebugMsg (TOPIC_JA2,DBG_LEVEL_3,"GameLoop: screen changed");
AssertNotNIL (GameScreens[guiCurrentScreen].HandleScreen);
+ UINT32 time = GetClock();
uiOldScreen = (*(GameScreens[guiCurrentScreen].HandleScreen))();
-
+ time = GetClock()-time;
+ if (time>1000)
+ ScreenMsg( FONT_MCOLOR_LTYELLOW, 0, L"Handle Screen completed in %d",time);
// if the screen has changed
if( uiOldScreen != guiCurrentScreen )
{
Index: Strategic/Map Screen Interface Map Inventory.cpp
===================================================================
--- Strategic/Map Screen Interface Map Inventory.cpp (revision 6401)
+++ Strategic/Map Screen Interface Map Inventory.cpp (working copy)
@@ -798,7 +798,8 @@
// Flugente: certain features need to alter an item's temperature value depending on the time passed
// if we do these functions here and adjust for the time passed since this sector was loaded last, it will seem to the player
// as if these checks are always performed in any sector
- SectorInventoryCooldownFunctions(sSelMapX, sSelMapY, ( INT16 )( iCurrentMapSectorZ ));
+ //Moa: removed this function and stripped the relevant code into BuildStashForSelectedSector to decrease loading time
+ //SectorInventoryCooldownFunctions(sSelMapX, sSelMapY, ( INT16 )( iCurrentMapSectorZ ));

// build stash
BuildStashForSelectedSector( sSelMapX, sSelMapY, ( INT16 )( iCurrentMapSectorZ ) );
@@ -1869,8 +1870,11 @@

uiTotalNumberOfSeenItems = uiItemCount;

+ //Flugente: update Temperature & Food (Moa: moved from SectorInventoryCooldownFunctions)
+ HandleSectorCooldownFunctions( sMapX, sMapY, (INT8)sMapZ, pTotalSectorList, uiTotalNumberOfSeenItems, TRUE );
+ SetLastTimePlayerWasInSector( sMapX, sMapY, (INT8)sMapZ );

- // now allocate space for all the unseen items
+ // now allocate space for all the unseen items
if( guiNumWorldItems > uiItemCount )
{
pUnSeenItems = new WORLDITEM[ guiNumWorldItems - uiItemCount ];
@@ -1912,6 +1916,9 @@
// now load into mem
LoadWorldItemsFromTempItemFile( sMapX, sMapY, ( INT8 ) ( sMapZ ), pTotalSectorList );

+ //Flugente: update Temperature & Food (Moa: moved from SectorInventoryCooldownFunctions)
+ HandleSectorCooldownFunctions( sMapX, sMapY, (INT8)sMapZ, pTotalSectorList, uiTotalNumberOfRealItems, TRUE );
+ SetLastTimePlayerWasInSector( sMapX, sMapY, (INT8)sMapZ );
}


@@ -5223,8 +5230,9 @@
}
}

+//Moa:moved relevant code of this function to BuildStashForSelectedSector and enterSector to increase loading time
// Flugente: handle various cooldown functions in a sector
-void SectorInventoryCooldownFunctions( INT16 sMapX, INT16 sMapY, INT16 sMapZ )
+/*void SectorInventoryCooldownFunctions( INT16 sMapX, INT16 sMapY, INT16 sMapZ )
{
UINT32 uiTotalNumberOfRealItems = 0;
WORLDITEM * pTotalSectorList = NULL;
@@ -5270,9 +5278,9 @@

//Save the Items to the the file
SaveWorldItemsToTempItemFile( sMapX, sMapY, (INT8)sMapZ, uiTotalNumberOfRealItems, pTotalSectorList );
-}
+}*/

-// Flugente: handle various cooldwon functions over an array of items in a specific sector.
+// Flugente: handle various cooldown functions over an array of items in a specific sector.
// if fWithMinutes = true, adjust cooldown for time since sector was last entered
// otherwise its used for a turn-precise cooldown
void HandleSectorCooldownFunctions( INT16 sMapX, INT16 sMapY, INT8 sMapZ, WORLDITEM* pWorldItem, UINT32 size, BOOLEAN fWithMinutes )
Index: Strategic/strategicmap.cpp
===================================================================
--- Strategic/strategicmap.cpp (revision 6401)
+++ Strategic/strategicmap.cpp (working copy)
@@ -2883,6 +2883,8 @@
#endif
}

+extern void SetLastTimePlayerWasInSector(INT16 sMapX, INT16 sMapY, INT8 sMapZ);
+//Moa: 09-14-2013 this function modifies the temperature of items in that sector as well \calls HandleSectorCooldownFunctions
BOOLEAN EnterSector( INT16 sSectorX, INT16 sSectorY , INT8 bSectorZ )
{
INT32 i;
@@ -3015,8 +3017,15 @@
// Flugente: certain features need to alter an item's temperature value depending on the time passed
// if we do these functions here and adjust for the time passed since this sector was loaded last, it will seem to the player
// as if these checks are always performed in any sector
- SectorInventoryCooldownFunctions(sSectorX, sSectorY, bSectorZ);
+ //Moa: removed this function and replaced by the handling function. Since we have allready loaded the items previously we can use the globals here.
+ //SectorInventoryCooldownFunctions(sSectorX, sSectorY, bSectorZ);
+ //moved from SectorInventoryCooldownFunctions, unvisible items are cooled down here as well. Since we are entering the sector this should not be a problem(?)
+ HandleSectorCooldownFunctions( sSectorX, sSectorY, (INT8)bSectorZ, gWorldItems, guiNumWorldItems, TRUE );
+ SetLastTimePlayerWasInSector( sSectorX, sSectorY, (INT8)bSectorZ );

+ //Moa: scatter items which have not yet been placed on the map
+ //ScatterUnplacedItems();
+
return TRUE; //because the map was loaded.
}