Details

    • Type: Task
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Release tasks
    • Labels:
      None
    • Affects:
      Documentation (Ref Guide, User Guide, etc.)
    • Estimated Difficulty:
      Low

      Description

      For reasons people can no longer even remember, Seam uses a very non-conventional code style. This complicates patches, impacts documentation and generally swims upstream against Java conventions.

      By adopting the JBoss Community coding style (specifically the code style used in AS 7 [1], one of the newest projects at JBoss), it simplifies things on the one hand while providing the appearance of JBoss projects working together on the other.

      This change would be made in the process of preparing a candidate release of the Seam stack. We'll also need to update the reference guides.

      The JBoss convention closely parallels the Java convention with a few exceptions. We believe that these exceptions are corrections to poor choices in the Java convention, so we could call it the Java convention 2.0. Those differences are as follows:

      AS 7 vs Java convention

      1. Spaces vs mixed tabs and spaces (a strange default)
      2. Single block indent for continued lines vs double block indent for continued lines
      3. 128 line width vs 80
      4. indent switch statements vs don't indent switch statements
      5. indent size 4 vs indent size 8
      6. minor javadoc differences

      [1] https://github.com/jbossas/jboss-as/raw/master/ide-configs/eclipse/as7formatter.xml

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                dan.j.allen Dan Allen
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: