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 244174 - Renaming will erase some comments
Summary: Renaming will erase some comments
Status: RESOLVED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Source (show other bugs)
Version: 8.0
Hardware: All All
: P3 normal (vote)
Assignee: Svata Dedic
URL:
Keywords:
Depends on: 239487
Blocks:
  Show dependency tree
 
Reported: 2014-04-29 14:29 UTC by crosati
Modified: 2014-07-10 14:28 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
IDE log (97.50 KB, text/plain)
2014-04-29 14:29 UTC, crosati
Details

Note You need to log in before you can comment on or make changes to this bug.
Description crosati 2014-04-29 14:29:40 UTC
Product Version = NetBeans IDE 8.0 (Build 201403101706)
Operating System = Mac OS X version 10.9.2 running on x86_64
Java; VM; Vendor = 1.8.0_05
Runtime = Java HotSpot(TM) 64-Bit Server VM 25.5-b02

Hi,

I have the following class:


    public class Metadata {

        ...

        @SuppressWarnings( "PublicInnerClass" )
        public static class Organization {

            public enum Keys {
                ...
            }

            private Organization () {
            }

            public static String metadataForName ( Keys metadataName, OrganizationDTO organization ) {
                ...
            }

        }   //  class Organization
    
    }   //  class Metadata


now if I rename the Keys enum, the "// class Organization" comment at the end of the Organization class definition will disappear.
Comment 1 crosati 2014-04-29 14:29:45 UTC
Created attachment 147006 [details]
IDE log
Comment 2 crosati 2014-04-29 14:38:44 UTC
Obviously renaming was performed using the Refactor->Rename… menu command.
Comment 3 Miloslav Metelka 2014-05-29 08:15:26 UTC
Reassigning to java/refactoring.
Comment 4 Ralph Ruijs 2014-05-30 12:24:40 UTC
diffTrailingComments() returns the localpointer, as there is no change, but this is ignored by diff() and diffTree(). Reassigning to Java/Source, please evaluate.
Comment 5 Svata Dedic 2014-06-06 14:09:00 UTC
still valid in the dev version
Comment 6 Svata Dedic 2014-07-10 14:28:15 UTC
Fixed by general support implemented as part of issue #239487 and isssue #245348