Uploaded image for project: 'OpenJPA'
  1. OpenJPA
  2. OPENJPA-1015

Enforce 80-column line width for source code

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Test
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0-M2
    • None
    • None
    • Patch Available

    Description

      There used to be a rule of 80-column width for source code.
      That rule is slowly yielding.

      Should we add a test case to catch such violation?
      Or should we not reignite the age-old battle about line width, placement of brackets etc?

      Attachments

        1. line80.txt
          17 kB
          Pinaki Poddar
        2. OPENJPA-1015-audit-output.patch
          0.5 kB
          Donald Woods
        3. OPENJPA-1015-checkstyle.xml.patch
          2 kB
          Donald Woods
        4. OPENJPA-1015-max80chars.patch
          262 kB
          Barrett Jay Reed
        5. OPENJPA-1015-newlineendoffile.patch
          13 kB
          Barrett Jay Reed
        6. OPENJPA-1015-plugin-version.patch
          0.6 kB
          Donald Woods
        7. OPENJPA-1015-test.patch
          585 kB
          Barrett Jay Reed
        8. TestLineWidth.java
          5 kB
          Pinaki Poddar

        Issue Links

        relates to

        Sub-task - The sub-task of the issue OPENJPA-832 Add eclipse formatting preferences to website

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Closed

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            bjreed Barrett Jay Reed
            ppoddar@apache.org Pinaki Poddar
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment