Commit Graph

4 Commits

Author SHA1 Message Date
jendave
8f2dfd1ddc resolved the excessive "TreeProperties returns null for...", it was a recursive call where null is okay and frequent 2011-08-06 04:01:33 +00:00
jendave
7c23c0bb3f Big change that should solve the memory leak
-replaced the old ImageCache class with ImageCache2
-extracted the card picture- and detail code from GuiDisplay/DeckEditor into separate classes (in forge.gui.game). therefore, CardDetailUtil and forge.CardPanel were deleted. GuiDisplayUtil is no longer used to create the card panels.

TESTS:
-confirmed all changed GUIs (but the quest deck editor - changes are the same as normal deck editor) to show details and pictures.
-I played a game with reduced heap size (-Xmx200M). Occasionally the image cache size shrinked along with the used memory.
-In a game without reduced heap size (memory usage ~500M, over 1000 cached images), i noticed slowdown when some images were missing.
-I noticed the stack elements now having a border with the card's color; this doesn't hurt IMO.

NOT WORKING
I wasn't able to adapt one thing: GUI_PictureHQ, i just didn't know what it does. for everything else to work, some code sections were commented out and tagged with "TODO reenable"
2011-08-06 03:25:05 +00:00
jendave
5c6efdf625 some small fixes and tweaks regarding the property system 2011-08-06 02:45:46 +00:00
jendave
e0224bd465 2011-08-06 02:37:41 +00:00