Avoid creating new unit selection objects.

This means always copy by reference when creating new frames. The
SelectionTracker can manually manage copies when it is changing a
buffer.

Author: GraylinKim

sc2reader/plugins/replay.py

@@ -111,6 +111,7 @@ def SelectionTracker(replay):
             selection = player_selection[event.frame]
 
             if isinstance(event, SelectionEvent):
+                selection[event.bank] = selection[event.bank].copy()
                 error = not selection[event.bank].deselect(*event.deselect)
                 selection[event.bank].select(event.objects)
                 if debug: logger.info("[{0}] {1} selected {2} units: {3}".format(Length(seconds=event.second),person.name,len(selection[0x0A].objects),selection[0x0A]))
@@ -127,6 +128,7 @@ def SelectionTracker(replay):
                 if debug: logger.info("[{0}] {1} set hotkey {2} to current selection".format(Length(seconds=event.second),person.name,event.hotkey))
 
             elif isinstance(event, AddToHotkeyEvent):
+                selection[event.hotkey] = selection[event.hotkey].copy()
                 error = not selection[event.hotkey].deselect(*event.deselect)
                 selection[event.hotkey].select(selection[0x0A].objects)
                 if debug: logger.info("[{0}] {1} added current selection to hotkey {2}".format(Length(seconds=event.second),person.name,event.hotkey))

sc2reader/plugins/utils.py

@@ -119,5 +119,5 @@ def __init__(self):
     def copy(self):
         new = PlayerSelection()
         for bank, selection in self.iteritems():
-            new[bank] = UnitSelection(selection.objects[:])
+            new[bank] = selection #UnitSelection(selection.objects[:])
         return new