You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently I have the issue where showing the ZynAddSubFX plugin caused lmms to crash. When inspecting this issue with QtCreator and trying different Qt versions (even Qt5), I discovered that it crashed due to the deletion of m_ioEngine.
When inspecting the stack trace, the crash occurred because the deletion triggered the QObject destructor. Somehow AudioOut and/or MidiIn became a subclass of the global Engine which is in charge of keeping the lmms state, and not a subclass of the Engine defined in Nio/Engine.h. I decided to put the lmms engine in a namespace to make it hidden to NulEngine which then solved the issue.
It may be useful to check for name clashes or prevent them in advance by defining namespaces for different parts of lmms. I will look if I can make my current changes a bit more clean and perhaps define a namespace for the Nio/ classes rather than the lmms classes (given that more depend on lmms Engine than the nio Engine)
The text was updated successfully, but these errors were encountered:
Currently I have the issue where showing the ZynAddSubFX plugin caused lmms to crash. When inspecting this issue with QtCreator and trying different Qt versions (even Qt5), I discovered that it crashed due to the deletion of m_ioEngine.
When inspecting the stack trace, the crash occurred because the deletion triggered the QObject destructor. Somehow AudioOut and/or MidiIn became a subclass of the global Engine which is in charge of keeping the lmms state, and not a subclass of the Engine defined in Nio/Engine.h. I decided to put the lmms engine in a namespace to make it hidden to NulEngine which then solved the issue.
It may be useful to check for name clashes or prevent them in advance by defining namespaces for different parts of lmms. I will look if I can make my current changes a bit more clean and perhaps define a namespace for the Nio/ classes rather than the lmms classes (given that more depend on lmms Engine than the nio Engine)
The text was updated successfully, but these errors were encountered: