From: Frederick W. <fre...@go...> - 2012-02-23 06:48:54
|
Stefan: > To 99% this must have been something to do with Frederick's change of > the NDC mechanism. Having taken a look at the online git repos, I confirm this. But since I do not have access to my local git clone for the next days, please feel free to push a fix to this according to the following analysis: (1) Game's add to the report buffer is called in Game's constructor but the new GameManager instance is only created in Game's setup method (called later). (2) ReportBuffer retrieves the game manager instance (yielding the old one). Before the NDC change, it was returned null and would buffer the added message. Two options to fix this: (1) Postpone the ReportBuffer add to Game's setup (after the new game manager is initialized). But from static analysis, it's unclear whether follow-up issues would arise. (2) Clear game manager singleton instance in game's constructor (would require an additional method to GameManager calling "gameManagerMap.put(GM_KEY, null);") -- Frederick |