Ticket #1869 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Add virtual to destructors

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: minor Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

Eclipse is complaining that a few classes have: virtual method 'category' but non-virtual destructor. These should be fixed.

Change History

comment:1 Changed 10 years ago by Michael Reuter

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Michael Reuter

(In [7231]) Adding virtual to destructors in Nexus package. This refs #1869.

comment:3 Changed 10 years ago by Michael Reuter

(In [7232]) Adding virtual to destructors in DataHandling package. This refs #1869.

comment:4 Changed 10 years ago by Michael Reuter

(In [7234]) Adding virtual to destructors in PythonAPI package. This refs #1869.

comment:5 Changed 10 years ago by Michael Reuter

  • Status changed from accepted to verify
  • Resolution set to fixed

All complaints fixed.

comment:6 Changed 10 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester set to Michael Whitty

comment:7 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

would appear to be true. eclipse is a whiney little thing, isn't it?

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2716

Note: See TracTickets for help on using tickets.