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

> 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
> 


-- 
Anssi Hannula

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

Reply via email to