clang-tidy: Avoid warnings for calling virtual methods from ctor
authorManuel Nickschas <sputnick@quassel-irc.org>
Tue, 4 Sep 2018 22:37:25 +0000 (00:37 +0200)
committerManuel Nickschas <sputnick@quassel-irc.org>
Thu, 6 Sep 2018 06:02:07 +0000 (08:02 +0200)
commit03f61d2ab68356bd74f6f014651c823e79678cbd
tree9ed5d784be7ad43deaa042b29ce61c1711c44854
parent895816988e0d03e30f4d54bf235af8d201aaad67
clang-tidy: Avoid warnings for calling virtual methods from ctor

MessageFilter and its derivatives call their idString() method
during construction in order to access their settings. This is fine,
but causes (legitimate) warnings in static analysis.

Make the scope explicit when calling idString() to avoid the warning,
and make it clear to readers that we're not interested in the derived
version.
src/client/messagefilter.cpp
src/client/messagefilter.h
src/qtui/chatmonitorfilter.cpp
src/qtui/chatmonitorfilter.h