Dimension3D icon indicating copy to clipboard operation
Dimension3D copied to clipboard

Is abstract but has non-virtual destructor

Open Lecrapouille opened this issue 2 years ago • 2 comments

Bonjour, il manque des virtual ~xxx() = default;

/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:38:3: warning: delete called on 'dim::Controller' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                delete controller;
                ^
/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:41:3: warning: delete called on 'dim::Camera' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                delete camera;
                ^
/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:53:3: warning: delete called on 'dim::Controller' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                delete controller;
                ^
/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:55:3: warning: delete called on 'dim::Camera' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                delete camera;
                ^
/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:200:3: warning: delete called on 'dim::Camera' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                delete this->camera;
                ^
/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:215:3: warning: delete called on 'dim::Controller' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                delete this->controller;
                ^
/Users/runner/work/Janus/Janus/external/Dimension3D/sources/windows/Scene.cpp:330:4: warning: delete called on 'dim::Light' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
                        delete light;
                        ^

Lecrapouille avatar Feb 05 '24 22:02 Lecrapouille

Possible de faire une PR ?

angeluriot avatar May 07 '24 11:05 angeluriot

PR opened. Tested some time ago on a fork of your Galaxy simulation https://github.com/Lecrapouille/Janus (also compiled on Mac OS X with CI until I realized that it does run because OpenCL uses OpenGL 4 that is not supported by Mac)

Lecrapouille avatar May 07 '24 16:05 Lecrapouille