Code review comment for lp:~gerboland/miral/no-unsigned-int-index
Revision history for this message
| Gerry Boland (gerboland) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
7020100
(Get the code!)
> + void windowAdded(const qtmir::WindowInfo, const int index);
>
> You do know that top level const is meaningless in [member] function
> declarations?
Yeah, I do. It is a habit, I tend to use const as much as possible, as even if the compiler cannot use it, it might hint to developer how api is intended to be used. How useful that actually is, I cannot say. Welcome to remove them if they annoy you.