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?


Thanks. I'll change the Makefiles accordingly.


vdr mailing list

Reply via email to