Goya in kdereview

Rafael Fernández López ereslibre at kde.org
Wed May 7 02:16:32 BST 2008


Hi,

Thanks for your checks Allen. I will tomorrow commit the Mainpage.dox.

> 1. Check for undocumented methods...  OOPS! 8 issues found!
>         kwidgetitemdelegate.h: 117, 138 (2)
>         kwidgetitemdelegatepool.cpp: 44, 46,47, 49, 50, 51 (6)

I can't find any method on kwidgetitemdelegate.h: 117 or 138. 117 is the 
beginning of a multi-line comment, and 138 is the d-pointer declaration.

> 2. Check for undocumented parameters...  OOPS! 1 issues found!
>         kwidgetitemdelegate.h: 128 (1)

Yeah, small one, it was written 'type' instead of 'types'. Thanks for that 
one.

> 3. Check for unknown parameters...  OOPS! 1 issues found!
>         kwidgetitemdelegate.h: 117 (1)

Same problem, our beloved bot couldn't find 'type' anywhere.

> 4. Check for methods that are undeclared or undefined...  OOPS! 9 issues
> found!
>         kwidgetitemdelegate.cpp: 62, 70, 154, 167, 176, 225, 260 (7)
>         kwidgetitemdelegatepool.cpp: 60, 70 (2)
>
> 7. Check for mismatched class members...  OOPS! 2 issues found!
>         kwidgetitemdelegate.cpp: 58 (1)
>         kwidgetitemdelegatepool.cpp: 56 (1)

Are those important ? I mean, they belong to the internal implementation.


Regards,
Rafael Fernández López.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080507/3874e868/attachment.sig>


More information about the kde-core-devel mailing list