Uploaded image for project: 'Commons Email'
  1. Commons Email
  2. EMAIL-10

[email][PATCH] Resolve checkstyle errors

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • None
    • None
    • Operating System: other
      Platform: All

    • 34918

    Description

      This patch includes trivial changes to clean up the Checkstyle reports of
      Commons Email. Changes include modifying or adding missing javadoc, cleaning
      indentation, curly braces, parenthesis, and removing tabs and trailing spaces at
      the end of lines.

      After applying this patch, Email.java contains two Checkstyle errors because of
      its use of Hashtable's instead of Maps's. The 8 remaining errors indicate that
      each file does not end with a newline, which they do. I assume this is an error
      with the Maven-Checkstyle plugin.

      Thanks,
      Eric Spiegelberg

      Attachments

        1. ASF.LICENSE.NOT.GRANTED--checkstyle_clean_up.patch
          28 kB
          Eric Spiegelberg
        2. ASF.LICENSE.NOT.GRANTED--big_checkstyle.patch
          252 kB
          Ramiro Pereira de Magalhaes

        Activity

          People

            Unassigned Unassigned
            lists@spiegs.com Eric Spiegelberg
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: