Code review comment for lp:~mterry/libdbusmenu-qt/dont-show-more-icons-than-desired

Revision history for this message
Aurélien Gâteau (agateau) wrote :

Hi Michael,

The patch looks good, it only need some coding style fixes:
- 4-space indents
- curly braces around if and else blocks

Thanks!

review: Needs Fixing

« Back to merge proposal