You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@netbeans.apache.org by "Reema Taneja (JIRA)" <ji...@apache.org> on 2018/06/06 10:31:00 UTC

[jira] [Commented] (NETBEANS-892) Refactor -> Move Class into other class breaks multicatches and lambas in destination class

    [ https://issues.apache.org/jira/browse/NETBEANS-892?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16503104#comment-16503104 ] 

Reema Taneja commented on NETBEANS-892:
---------------------------------------

Tried with jdk8 +nb-javac plugin.

Found below issues :
1 In MoveTo.java 

catch stmt in a method changed from 

catch (RuntimeException | AssertionError e){
to
catch (RuntimeException | AssertionErrore){

(Compilation error, issue reproducible in NetBeans 8.2)

2 In MoveTo.java,
Method call with Lambda expr got modified from
doStuff(item-> item.substring(20));
to 

doStuff(String item-> item.substring(20));
(Compilation error, issue reproducible in NetBeans 8.2)

3 In MoveFrom.java, class unrelated to above Move change, same issues as 1 and 2 above.

4 In ReadOnlyCode.java, original file had no imports, class used package appended Class names. Move operation added imports. This seems to be the only change.
No Compilation issue. But unrelated class shouldn't have got modified.

(Issues 3, 4 are not reproducible in NetBeans 8.2)



> Refactor -> Move Class into other class breaks multicatches and lambas in destination class
> -------------------------------------------------------------------------------------------
>
>                 Key: NETBEANS-892
>                 URL: https://issues.apache.org/jira/browse/NETBEANS-892
>             Project: NetBeans
>          Issue Type: Bug
>          Components: java - Refactoring
>    Affects Versions: 9.0
>            Reporter: Austin Stephens
>            Assignee: Reema Taneja
>            Priority: Blocker
>             Fix For: 9.0
>
>         Attachments: EvilRefactorBug.zip
>
>
> Move the class "MoveMeIn" into the class "MoveTo". Things break. Many, many thing break.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@netbeans.apache.org
For additional commands, e-mail: commits-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists