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.
Summary: | Bad import fix if unused static exists | ||
---|---|---|---|
Product: | java | Reporter: | fommil <fommil> |
Component: | Editor | Assignee: | Max Sauer <msauer> |
Status: | CLOSED WORKSFORME | ||
Severity: | blocker | CC: | jkovalsky |
Priority: | P3 | ||
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: |
Description
fommil
2009-04-20 18:31:26 UTC
Hi Sam, could you please provide sample source and your build number? I cannot reproduce this issue with: package javaapplication9; import java.util.List; import static ss.NewClass.getSS; import java.util.Set; public class Main { Set s; List l; } ---- Product Version: NetBeans IDE Dev (Build 090414) Java: 1.5.0_16; Java HotSpot(TM) Client VM 1.5.0_16-133 System: Mac OS X version 10.5.6 running on i386; MacRoman; en_US (nb) I'd like to NetFIX [1] this bug. Is it possible? [1] http://wiki.netbeans.org/NetFIX Yes, you can -- and you're welcome. But first mention a reproducible scenario in this issue. Odd, I've not been able to reproduce... although it happened to me twice today. Closing for now, but will reopen when I can isolate the offending actions! Having a reproducible test case, this should not be very hard to fix. Most likely a bug in the code generator. Bugs in code generator are typically fixed as follows: -add one or more tests into the generator test suite (package org.netbeans.api.java.source.gen in java.source's tests). In this case, the test(s) should be added into ImportsTest, IMO. -fix the problem, so that the test passes. In this case, most likely in CasualDiff in java.source. -run whole codegen test suite by executing "ant -Dtest.config=generator test" inside java.source module. All tests must pass (this test suite is being run during the jet-main continuous build, so all tests should be passing). I can't reproduce... happy for you to close. |