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

Fix several findbugs issues

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      This issue is to resolve the following findbug issues:

      • Use %n instead of \n to handle platform-specific line separators.
      • Use StringBuilder for concatenating strings.
      • Unnecessary unboxing of longs.
      • Ignore the debugging static variable which can be mutable.

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Steven Christou
          Path:
          src/main/java/hudson/scm/RevisionParameterAction.java
          src/main/java/hudson/scm/SubversionChangeLogBuilder.java
          http://jenkins-ci.org/commit/subversion-plugin/0c52b546082dafbced41feaf7f74ad3158b96efb
          Log:
          Merge pull request #123 from christ66/findbugs-1

          JENKINS-29492 Fix some findbugs issues

          Compare: https://github.com/jenkinsci/subversion-plugin/compare/8e1e0aeefb97...0c52b546082d

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Steven Christou Path: src/main/java/hudson/scm/RevisionParameterAction.java src/main/java/hudson/scm/SubversionChangeLogBuilder.java http://jenkins-ci.org/commit/subversion-plugin/0c52b546082dafbced41feaf7f74ad3158b96efb Log: Merge pull request #123 from christ66/findbugs-1 JENKINS-29492 Fix some findbugs issues Compare: https://github.com/jenkinsci/subversion-plugin/compare/8e1e0aeefb97...0c52b546082d
          Hide
          recena Manuel Recena Soto added a comment - - edited

          The changes related to this ticket was been merged and included in Subversion Plugin 2.5.2

          Show
          recena Manuel Recena Soto added a comment - - edited The changes related to this ticket was been merged and included in Subversion Plugin 2.5.2

            People

            • Assignee:
              schristou Steven Christou
              Reporter:
              schristou Steven Christou
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: