[kepler-dev] about org.kepler.tagging.gui.TagBarUI class.

[Jianwu Wang]

  Hi Sean,

     Cool. I'll do it.

Best wishes

Sincerely yours

Jianwu Wang
jianwu at sdsc.edu
http://users.sdsc.edu/~jianwu/

Assistant Project Scientist
Scientific Workflow Automation Technologies (SWAT) Laboratory
San Diego Supercomputer Center
University of California, San Diego
San Diego, CA, U.S.A.


On 9/24/2010 12:24 PM, Sean Riddle wrote:
> If it's not being used, which it probably isn't, go right head. Sounds 
> like I just put that there expecting I would need it, and then didn't. 
> And it definitely screams memory leak.
>
> - Sean
>
> On Sep 24, 2010, at 11:53 AM, Jianwu Wang wrote:
>
>> Hi Sean,
>>
>>   I've been working on kepler memory leak fixing and got into a 
>> question to the class org.kepler.tagging.gui.TagBarUI.
>>
>>   It has a variable: final static Map<Integer, TagBarUI> tagBarUIs.  
>> It put a new element in its map whenever a new TagBarUI object is 
>> initiated. It's never used other places. From JProfile, I can see it 
>> still hold reference to KeplerGraphFrame object after the window is 
>> closed. I tried to change the type to be WeakHashMap, but it won't work.
>>
>>   I suggest to remove this variable. What do you think?
>>
>>
>> -- 
>>
>> Best wishes
>>
>> Sincerely yours
>>
>> Jianwu Wang
>> jianwu at sdsc.edu
>>
>> Assistant Project Scientist
>> Scientific Workflow Automation Technologies (SWAT) Laboratory
>> San Diego Supercomputer Center
>> University of California, San Diego
>> San Diego, U.S.A.
>>