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 29001 - Node forces cookie set initialization
Summary: Node forces cookie set initialization
Status: VERIFIED WONTFIX
Alias: None
Product: platform
Classification: Unclassified
Component: Nodes (show other bugs)
Version: 3.x
Hardware: Sun All
: P3 blocker (vote)
Assignee: Petr Hrebejk
URL:
Keywords: PERFORMANCE
Depends on:
Blocks: 28623
  Show dependency tree
 
Reported: 2002-11-25 19:39 UTC by _ tboudreau
Modified: 2008-12-22 23:15 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
1 line patch eliminating the call to allItems() (642 bytes, patch)
2002-11-25 19:40 UTC, _ tboudreau 		Details | Diff
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 _ tboudreau 2002-11-25 19:39:39 UTC 
Node.LookupEventList.init() calls allItems()
unnecessarily, forcing things to be instantiated
that don't need to be.
Comment 1 _ tboudreau 2002-11-25 19:40:34 UTC 
Created attachment 8045 [details]
1 line patch eliminating the call to allItems()
Comment 2 _ tboudreau 2003-01-14 12:24:27 UTC 
FYI, I've been running my development copy of NB with this patch
for several months with no problem.  Any reason not to integrate
it?  It looks like the call to allItems() was probably test code
someone forgot to delete, and it does force instantiation of a bunch
of things that may never be used.
Comment 3 Petr Hrebejk 2003-01-28 13:39:16 UTC 
According to the automated tests this is wrong. Should you run tests
on your patch you will immediately see that NodeLookupTests starts
failing.
Comment 4 Marian Mirilovic 2003-07-28 15:48:29 UTC 
verifying.