Skip to content

Commit

Permalink
tidy: Fix virtual calls from object constructor/destructor.
Browse files Browse the repository at this point in the history
Clang-tidy complains about a class calling a virtual function from
its constructor or destructor.  During constructors and destructors
virtual functions aren't virtual, calling the function in that class
and not any derived class.  Add the class name to these calls to
eliminate the warning messages.

https://clang.llvm.org/extra/clang-tidy/checks/clang-analyzer-optin.cplusplus.VirtualCall.html
  • Loading branch information
linuxdude42 committed Dec 25, 2019
1 parent a5aba64 commit bd0291e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion mythtv/libs/libmythbase/platforms/mythpowerdbus.cpp
Expand Up @@ -61,7 +61,7 @@ MythPowerDBus::MythPowerDBus()
{
m_delayTimer.setSingleShot(true);
connect(&m_delayTimer, &QTimer::timeout, this, &MythPowerDBus::ReleaseLock);
Init();
MythPowerDBus::Init();
}

MythPowerDBus::~MythPowerDBus()
Expand Down
2 changes: 1 addition & 1 deletion mythtv/libs/libmythtv/opengl/mythnvdecinterop.cpp
Expand Up @@ -26,7 +26,7 @@ MythNVDECInterop::MythNVDECInterop(MythRenderOpenGL *Context)
MythNVDECInterop::~MythNVDECInterop()
{
m_referenceFrames.clear();
DeleteTextures();
MythNVDECInterop::DeleteTextures();
CleanupContext(m_context, m_cudaFuncs, m_cudaContext);
}

Expand Down
2 changes: 1 addition & 1 deletion mythtv/libs/libmythtv/opengl/mythvaapidrminterop.cpp
Expand Up @@ -89,7 +89,7 @@ void MythVAAPIInteropDRM::DestroyDeinterlacer(void)
if (m_filterGraph)
{
LOG(VB_PLAYBACK, LOG_INFO, LOC + "Deleting deinterlacer frame cache");
DeleteTextures();
MythVAAPIInteropDRM::DeleteTextures();
CleanupDRMPRIME();
}
MythVAAPIInterop::DestroyDeinterlacer();
Expand Down

0 comments on commit bd0291e

Please sign in to comment.