Code review comment for lp:~chrisccoulson/libdbusmenu/lp1011073
Revision history for this message
| Charles Kerr (charlesk) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Besides the fact that "&=" is an unusual way to deal with the boundary case, it's also inconsistent with the more conventional % implementation at line 391.
Chris, this would be a great fix to get in, could you revise this?