will mason created NETBEANS-687:
-----------------------------------

             Summary: Code Refactoring Must respect code style, formatting, 
spacing and layout
                 Key: NETBEANS-687
                 URL: https://issues.apache.org/jira/browse/NETBEANS-687
             Project: NetBeans
          Issue Type: Bug
          Components: editor - Refactoring, java - Refactoring
    Affects Versions: 9.0
         Environment: Product Version: Apache NetBeans IDE Dev (Build 
incubator-netbeans-release-205-on-20180202)
Java: 10; Java HotSpot(TM) 64-Bit Server VM 10+46
Runtime: Java(TM) SE Runtime Environment 10+46
System: Windows 10 version 10.0 running on amd64; Cp1252; en_AU (nb)
User directory: Z:\tmp\.other\user\netbeans\v09.00-beta\FourAbs
Cache directory: Z:\tmp\.other\cache\netbeans\FourAbs-09
            Reporter: will mason
             Fix For: 9.0


h2. expected

*  I expect automatic code edits to respect code layout as much as possible
* For example the code layout below should NOT change, following...
* Rename of '{{jComboBoxCorridor}}'  --> '{{jComboBoxSectionCorridor}}'   
{code:java}
      //  Example 1
    helper.enable(
        jComboBoxNoteType,
        jComboBoxSectionCorridor
    );

        //  Example 2

    helper.enable
    (
        jComboBoxNoteType,
        jComboBoxSectionCorridor
    );

    // Example 3

    helper.setEnabled(
        jRadioButtonSection.isSelected() && enabled,
            jComboBoxSectionCorridor,
            jComboBoxNirCode
    );
{code}

h2. actual

*  This is highly undesirable
* The first two code examples are change to the following:  {code:java}
    helper.enable(jComboBoxNoteType,
        jComboBoxSectionCorridor
    );
{code}
* Example 3 results in a even less desirable change:  {code:java}
    helper.enable(
            (jRadioButtonSection.isSelected() && enabled,
            jComboBoxSectionCorridor,
            jComboBoxNirCode
{code}
**  In this case the indenting was showing that the boolean result is applied 
to all the indented fields.
**  That information has been lost and was overlooked during debugging 
* These changes are definitely potential source for application code bugs






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

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

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

Reply via email to