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 198335 - The "Replace Map with EnumMap" hint fix only partially works.
Summary: The "Replace Map with EnumMap" hint fix only partially works.
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: Hints & Annotations (show other bugs)
Version: 7.0
Hardware: PC Windows 7 x64
: P3 normal (vote)
Assignee: Jan Lahoda
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-05 06:36 UTC by matsa
Modified: 2011-08-14 07:25 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Screenshots showing the code state before and after applying the fix (50.28 KB, image/png)
2011-05-05 06:36 UTC, matsa 		Details
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 matsa 2011-05-05 06:36:26 UTC 
Created attachment 108126 [details]
Screenshots showing the code state before and after applying the fix

The "Replace Map with EnumMap" hint fix only partially works. While it adds the correct enum class argument to EnumMap's constructor, it neglects to actually swap out 'new HashMap' with 'new EnumMap'. The screenshot should illustrate this sufficiently.
Comment 1 Jan Lahoda 2011-05-05 15:21:21 UTC 
Good catch, thank you!

http://hg.netbeans.org/jet-main/rev/7373939a6913
Comment 2 Quality Engineering 2011-05-06 05:08:23 UTC 
Integrated into 'main-golden', will be available in build *201105060000* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/7373939a6913
User: Jan Lahoda <jlahoda@netbeans.org>
Log: #198335: Correcting the fix for HashMap->EnumMap conversion.