Add comments

zanata · Jun 22, 2012 · 572b73f · 572b73f
1 parent 4d5d42f
commit 572b73f
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/service/UserSessionService.java b/zanata-war/src/main/java/org/zanata/webtrans/client/service/UserSessionService.java
@@ -93,13 +93,16 @@ public void addUser(EditorClientId editorClientId, UserPanelSessionItem item)
    public void removeUser(EditorClientId editorClientId)
    {
       userSessionMap.remove(editorClientId);
+      // FIXME remove from colorListMap (memory leak)
    }
 
    public Map<EditorClientId, UserPanelSessionItem> getUserSessionMap()
    {
       return userSessionMap;
    }
 
+   // TODO what we pass is really editorClientId
+   // Should we be passing sessionId?  See also memory leak above.
    public String getColor(String sessionId)
    {
       if (colorListMap.containsKey(sessionId))