Groovy
  1. Groovy
  2. GROOVY-834

Many, many compiler warnings (Eclipse 3.1 M6)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0-JSR-1
    • Fix Version/s: 1.0-JSR-4
    • Component/s: None
    • Labels:
      None
    • Environment:
      Eclipse 3.1M6, using 1.4 compiler settings and enabled many warnings

      Description

      See the attached "\t" separated file. It lists around 700 warnings Eclipse 3.1M6 (using 1.4 compiler settings and enabled many warnings) generates. Which ones are no-gos? Which are to be ignored?

      I start a list here, sorting out the severity type of warning descriptions.

      [ might be ignored ] (by Eclipse's and other compiler settings)

      o Access to enclosing method getCurrent() from the type T is emulated by a synthetic accessor method. Increasing its visibility will improve your performance.

      o Statement unnecessarily nested within else clause. The corresponding then clause does not complete normally.

      [ needs fix ]

      o all remaining, especially...

      o Unnecessary semicolons
      o Unnecessary casts from T to T
      o The private field f is never read locally.

        Issue Links

          Activity

          sormuras created issue -
          Hide
          sormuras added a comment -

          Switching to 1.5 compiler yields 1700+ warnings and the 3 errors reported in http://jira.codehaus.org/browse/GROOVY-833 ... mainly complaining about type-safety. There's lurking another 2d+ work, when Groovy is "ported" to Java 5.

          Show
          sormuras added a comment - Switching to 1.5 compiler yields 1700+ warnings and the 3 errors reported in http://jira.codehaus.org/browse/GROOVY-833 ... mainly complaining about type-safety. There's lurking another 2d+ work, when Groovy is "ported" to Java 5.
          sormuras made changes -
          Field Original Value New Value
          Link This issue is related to GROOVY-833 [ GROOVY-833 ]
          Guillaume Delcroix made changes -
          Fix Version/s 1.0-JSR-4 [ 11905 ]
          Fix Version/s 1.0-JSR-3 [ 11518 ]
          Hide
          sormuras added a comment -

          Only 20 warnings left: 19 not needed import statements and 1 semicolon.
          Affected files are all generated by antlr - so it's more an antlr "problem".

          Show
          sormuras added a comment - Only 20 warnings left: 19 not needed import statements and 1 semicolon. Affected files are all generated by antlr - so it's more an antlr "problem".
          sormuras made changes -
          Assignee Christian Stein [ sormuras ]
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Paul King made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
          Mark Thomas made changes -
          Workflow jira [ 12729984 ] Default workflow, editable Closed status [ 12741971 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
          Mark Thomas made changes -
          Workflow jira [ 12971861 ] Default workflow, editable Closed status [ 12978951 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          194d 3h 44m 1 sormuras 15/Nov/05 03:50
          Resolved Resolved Closed Closed
          745d 2h 32m 1 Paul King 30/Nov/07 06:23

            People

            • Assignee:
              sormuras
              Reporter:
              sormuras
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 48h
                48h
                Remaining:
                Remaining Estimate - 48h
                48h
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Development