|
|
Am 17.08.2012 23:35, schrieb Dave:
> On 8/17/2012 4:17 PM, clipka wrote:
>>> vfesession.cpp:594:11: warning: case label value is less than minimum
>>> value for type [enabled by default]
>>
>> That line seems somewhat bogus.
>
> Variable 'm_BackendState' is declared as type 'State' (vfe/vfesession.h
> line 1316). The switch statement (vfe/vfesession.cpp line 576) contains
> a 'case -1:'. Whoever wrote that code perhaps expected that
> m_BackendState could have the value '-1'. If it does, then the case will
> never be caught since '-1' is not a valid value for that enum.
Yes, that's what I'm thinking. Have done some quick code digging to see
if it is ever set to -1 somewhere, but it turns out to be more than a
5-minute job.
Post a reply to this message
|
|