Code review comment for lp:~oif-team/frame/semi-mt-add-touches
Revision history for this message
| Henrik Rydberg (rydberg) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
a60fb26
(Get the code!)
Found two problems with the current code - lin e 163, missing setting of active variable, and in the identifier code, possible wrap of values, resulting in close by use of identifiers. In addition, I would prefer if the semi-mt code was broken out in its own branch.
I have created a merge request to this branch, fixing the problems above.