Uploaded image for project: 'FishEye'
  1. FishEye
  2. FE-2652

Show moved lines specially in diffs

    XMLWordPrintable

Details

    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

    Description

      When refactoring code, often several lines will get pulled out of one method and added to a new one or vice versa. In simple diff tools, these changes are frequently annotated as deletes and adds (or confoundingly with the intervening code marked up as added and deleted), but they can be shown as moved-from and moved-to instead. Showing them with this extra info would help reviewers ignore the code (if they trust the existing code), or to find where it got to (if they've found an existing bug and are checking if it got fixed). In the browser, these code motion annotations could be hyperlink anchors within the file to make it easy to jump to and from the old and new locations.

      Implementing this in the surrounding tools such as blame would also help track down the code long after any refactoring happened.

      Attachments

        Activity

          People

            Unassigned Unassigned
            richatkins Richard Atkins
            Votes:
            4 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: