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 - o.n.m.c.makeproject.api.configurations.Item shouldn't store file
Summary: o.n.m.c.makeproject.api.configurations.Item shouldn't store file
Alias: None
Product: cnd
Classification: Unclassified
Component: Project (show other bugs)
Version: 6.x
Hardware: Sun All
: P3 blocker (vote)
Assignee: Thomas Preisler
Depends on:
Blocks: 120036 123872
  Show dependency tree
Reported: 2007-10-25 14:45 UTC by Vladimir Kvashin
Modified: 2010-04-14 23:19 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Note You need to log in before you can comment on or make changes to this bug.
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?