groovy

Fix warnings

Details

  • Type: Wish Wish
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.0-beta-5
  • Component/s: None
  • Labels:
    None
  • Environment:
    Sun J2SDK 1.4.2, Eclipse 3.0M7
  • Number of attachments :
    0

Description

Turning on Warnings for unread variables and exceptions not actually thrown (Window->Preferences, Java->Compiler->Unused Code) gives 14 warnings in GroovyEclipse and 3 warnings in GroovyEclipseTest. Most of these are relevant.

Turning on Warnings for uncommented empty blocks (Window->Preferences, Java->Compiler) gives 15 more warnings, some of which are relevant (swallowed exceptions).

Even though none of these warnings indicate serious problems, it would be nice to have them resolved, so that developers can run Eclipse with these settings without being distracted or having real problems obscured by too many messages.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
james strachan added a comment -

tut tut!

Show
james strachan added a comment - tut tut!
Hide
Permalink
Zohar Melamed added a comment -

If this was IntelliJ I would have just spotted them being grey , or the file marker beiong yellow.

As it's eclipse it has to be complicated...

anyway, guilty as charged , will fix this.

Show
Zohar Melamed added a comment - If this was IntelliJ I would have just spotted them being grey , or the file marker beiong yellow. As it's eclipse it has to be complicated... anyway, guilty as charged , will fix this.

People

  • Assignee:
    Zohar Melamed
    Reporter:
    Johannes Brodwall
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: