Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-14496

No contextHashCode is set to annotations parsed from cpd

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: dry-plugin
    • Labels:
      None
    • Similar Issues:

      Description

      It looks like the CpdParser is missing a call to setContextHashCode. As a result, every annotation receives an autogenerated context hash, which is different on every run. This causes the delta between two identical cpd-files to be calculated incorrectly (all duplications are both fixed and new), as the differencer thinks the annotations from two consecutive parser runs are different due to a different context hash.

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/dry/DryPublisher.java
          src/main/java/hudson/plugins/dry/DryReporter.java
          src/main/java/hudson/plugins/dry/parser/AbstractDryParser.java
          src/main/java/hudson/plugins/dry/parser/DuplicationParserRegistry.java
          http://jenkins-ci.org/commit/dry-plugin/4066e2e8a345c4dacd1fd85340f0aac10d7ef03f
          Log:
          [FIXED JENKINS-14496] Set context hash code after parsing.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/dry/DryPublisher.java src/main/java/hudson/plugins/dry/DryReporter.java src/main/java/hudson/plugins/dry/parser/AbstractDryParser.java src/main/java/hudson/plugins/dry/parser/DuplicationParserRegistry.java http://jenkins-ci.org/commit/dry-plugin/4066e2e8a345c4dacd1fd85340f0aac10d7ef03f Log: [FIXED JENKINS-14496] Set context hash code after parsing.

            People

            • Assignee:
              drulli Ulli Hafner
              Reporter:
              aboks Arnout Boks
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: