Uploaded image for project: 'Daffodil'
  1. Daffodil
  2. DAFFODIL-2797

Make Runner final

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: In Progress
    • Trivial
    • Resolution: Unresolved
    • None
    • None
    • Clean Ups
    • 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.

      Attachments

        Activity

          People

            mbeckerle Mike Beckerle
            jinterrante John Interrante
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: