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
Status: RESOLVED FIXED
Alias: None
Product: cnd
Classification: Unclassified
Component: Project (show other bugs)
Version: 6.x
Hardware: Sun All
: P3 blocker (vote)
Assignee: Thomas Preisler
URL:
Keywords:
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:

Attachments
Add an attachment (proposed patch, testcase, etc.)

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?