Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-13385

Remove unnecessary virtual destructors

    XMLWordPrintable

    Details

    • Type: Story
    • Status: In Progress
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: afw
    • Story Points:
      1
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      In the course of the review of DM-9932, I noticed a number of classes with virtual destructors that probably don't need them (I think just due to the paranoia many new C++ developers have when they first learn about destructors; these are all very old classes). Fixing that is out of scope for DM-9932, but we should eventually look more carefully at each case and then remove them.

      The classes in question:

      • Exposure
      • ImagePca
      • ImageBase
      • Image
      • Mask
      • BackgroundControl

        Attachments

          Issue Links

            Activity

            jbosch Jim Bosch created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Remote Link This issue links to "Page (Confluence)" [ 31060 ]
            wittgen Matthias Wittgen made changes -
            Assignee Matthias Wittgen [ wittgen ]
            wittgen Matthias Wittgen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            wittgen Matthias Wittgen made changes -
            Team Architecture [ 10304 ]
            Urgent? off
            wittgen Matthias Wittgen made changes -
            Story Points 1

              People

              Assignee:
              wittgen Matthias Wittgen
              Reporter:
              jbosch Jim Bosch
              Watchers:
              Jim Bosch, Krzysztof Findeisen, Matthias Wittgen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins Builds

                  No builds found.