Opened 16 years ago
Closed 16 years ago
#257 closed defect (fixed)
get rid of all compiler warnings
| Reported by: | Stephen Siegel | Owned by: | zirkel |
|---|---|---|---|
| Priority: | major | Milestone: | 1.0.1 |
| Component: | multiple | Version: | 1.0.1 |
| Keywords: | warnings compiler | Cc: |
Description
Get rid of all compiler warnings in the TASS code.
front: Ben
model, dynamic, symbolic, state, verify: Steve
other: Tim
Change History (7)
comment:1 by , 16 years ago
comment:2 by , 16 years ago
I'm still seeing warnings in model and prove as well. Should I work on removing them? Or was something not committed?
comment:3 by , 16 years ago
What warnings do you see? Make sure you are updated. I'm on 1843 and see no warnings in those modules.
comment:4 by , 16 years ago
For example, in SimpleIdealProver:
line 70: Unnecessary @SuppressWarnings("unchecked")
line 71: HashMap is a raw type
Same errors for lines 57 and 58 of IdealCVC3HybridProver and lines 54 and 55 of TheoremProverIF
In Function and AbstractFunction, warnings about dead code.
comment:5 by , 16 years ago
| Owner: | set to |
|---|---|
| Status: | new → accepted |
comment:6 by , 16 years ago
The suggested way to get rid of the warnings for the ANTLR generated files (according to antlr.org) is to add an @SuppressWarnings("all") in the Java.stg template inside of antlrworks.jar. It's located at antlrworks/org/antlr/codegen/templates/Java. So far I don't see a way to handle it by modifying our build settings.
comment:7 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
All warnings are gone except for those in files that ANTLR generates. Removing these warnings would require changes to ANTLR, or at least to the ANTLR template files. We're not going to do that at this time.
Our policy will be to not introduce any new warnings in our code.

Warnings now gone from everywhere except front.
Whoever wants to work on removing warnings from front, please accept this ticket.