Suspicous code in kdelibs-3.5.2

[Thiago Macieira]

Lubos Lunak wrote:
>> -----------------------------------------------------------------
>> Cases from switch statements that fall through in some cases but
>> do not have a fall through comment as in most such cases.
>> ------------------------------------------------------------------
>>
>> - kdecore/kglobalaccel_x11.cpp:246
>
> Actually intended.

Is it intended to be confusing? :-)

This is what the test catches: switch statements that fall through (which 
is quite ok) without a comment saying "yes, fall through. I did not 
forget to break here".

-- 
Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
  thiago.macieira (AT) trolltech.com     Trolltech AS
    GPG: 0x6EF45358                   |  Sandakerveien 116,
    E067 918B B660 DBD1 105C          |  NO-0402
    966C 33F5 F005 6EF4 5358          |  Oslo, Norway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20060409/25de3232/attachment.sig>