Details
-
Improvement
-
Status: In Progress
-
Trivial
-
Resolution: Unresolved
-
None
-
None
-
None
Description
We have hundreds of Test*.scala files in Daffodil, each with at least one 'val runner' constructing a test suite from a TDML file. IDEA has warnings in all of these files saying that we need explicit type declarations for these runners. However, no user ever extends the Runner class so we could make Runner a final class, which would eliminate all of these warnings in IDEA. Scala has ways to let users extend even final classes like String so I am not worried about preventing users from doing something with Runner.
We also could look for other Daffodil classes which cause similar IDEA warnings and make them final too if they are unlikely to ever be subclassed.