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?

I'd hate to have to add that "final" keyword to all virtual function 
declarations...

Klaus

_______________________________________________
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr

Reply via email to