On 11.03.2012 16:05, Anssi Hannula wrote:
11.03.2012 16:59, Klaus Schmidinger kirjoitti:
On 11.03.2012 15:24, Udo Richter wrote:
Am 10.03.2012 16:18, schrieb Klaus Schmidinger:
Any plugins that implement cStatus::ChannelSwitch() need to add
the parameter 'bool LiveView' to that function.
The 'status' plugin, part of the VDR source, does not yet use the new LiveView parameter.
Yikes! Sorry about that.
Just one of many plugin breakages. People should look out for warnings like this:
../../../include/vdr/status.h:33: warning: ‘virtual void cStatus::ChannelSwitch(const cDevice*, int, bool)’ was hidden status.c:21: warning: by ‘virtual void cStatusTest::ChannelSwitch(const cDevice*, int)’
This usually means that channel switch reporting is no longer working!
People with C++11 compatible compilers can enable C++11 support and add another declaration like this to cStatus:
virtual void ChannelSwitch(const cDevice *Device, int
ChannelNumber) final;
That will cause the wrong parameters to be treated as an error.
Isn't there a way of making this a general error? Like -Woverloaded-virtual makes this a warning, could something like -Eoverloaded-virtual make it an error?
-Werror=overloaded-virtual
Thanks. I'll change the Makefiles accordingly.
Klaus