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 72889 - [Clean up] Commenting out does not work
Summary: [Clean up] Commenting out does not work
Status: NEW
Alias: None
Product: java
Classification: Unclassified
Component: Refactoring (show other bugs)
Version: 5.x
Hardware: PC Linux
: P2 blocker (vote)
Assignee: _ leonchiver
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-21 15:27 UTC by Jiri Prox
Modified: 2006-11-22 14:00 UTC (History)
0 users

See Also:
Issue Type: TASK
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jiri Prox 2006-02-21 15:27:03 UTC
NB 6.0 200602191900
JDK 1.5.0_06
Ex. refactoring: 0.35.0.2.3

Commenting out unused elements does not work.

Steps to reproduce:
1) create new class:

import java.util.Date;
public class Main {
    
    private int a;

    private void method() {        
    }

    public static void main(String[] args) {
        Date d = new Date();
    }    
}

2) call clean up on it
3) in dialog check Comment Out ...
4) confirm
-> in preview all unused elements are found and labeled 'Comment out'
5) confirm preview
-> the class is not modified

Normal clean up (e.g. with deleting) is OK
Comment 1 Jan Becicka 2006-02-22 15:05:04 UTC
Yes. This is not implemented so far. I disabled this feature and I added
explanatory tooltip:
Checking in Bundle.properties;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/ui/Bundle.properties,v
 <--  Bundle.properties
new revision: 1.34; previous revision: 1.33
done
Checking in CleanUpPanel.form;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/ui/CleanUpPanel.form,v
 <--  CleanUpPanel.form
new revision: 1.2; previous revision: 1.1
done
Checking in CleanUpPanel.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/ui/CleanUpPanel.java,v
 <--  CleanUpPanel.java
new revision: 1.3; previous revision: 1.2
done

Let's change this issue to TASK.
Leon, can you take a look at it?
Thanks
Comment 2 _ leonchiver 2006-02-23 06:18:58 UTC
Jan, can you tell me how your time constraints look? I probably won't be able to
do it for the next 2,3 weeks. Would that be OK with your schedule?
Comment 3 Jan Becicka 2006-02-23 08:50:00 UTC
Yes it is OK. This feature is not in final product so far.