This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 120042

Summary: o.n.m.c.makeproject.api.configurations.Item shouldn't store file
Product: cnd Reporter: Vladimir Kvashin <vkvashin>
Component: ProjectAssignee: Thomas Preisler <thp>
Status: RESOLVED FIXED    
Severity: blocker    
Priority: P3    
Version: 6.x   
Hardware: Sun   
OS: All   
Issue Type: DEFECT Exception Reporter:
Bug Depends on:    
Bug Blocks: 120036, 123872    

Description Vladimir Kvashin 2007-10-25 14:45:30 UTC
Now org.netbeans.modules.cnd.makeproject.api.configurations.Item 
has File field. This field is used for caching canonical path. 
Code model uses canonical paths for the same files too. 
So the proposal is to 
1) use string instead of File
2) share this strings between modules
(There is such string sharing machanizm in apt module; it should be extracted into a separate module)
Comment 1 Vladimir Kvashin 2007-10-25 16:08:30 UTC
I don't mean this will save a lot - probably about 1-2% of total memory consumption.
But, as soon as we start makeproject memory optimizations (I hope this will occur some day), this 1-2% will be a rather
low hanging fruit.
Comment 2 Thomas Preisler 2010-04-14 23:19:17 UTC
Not sure what to fix. It make sense to cache File in Item for internal purposes and File is available with getFile() in the NativeFileItem interface. Why can't you just use NativeFileItem.getFile() in your code?