Code review comment for lp:~therp-nl/therp-backports/addons-6.1_lp882036_account_rounding
Revision history for this message
| Ronald Portier (Therp) (rportier1962) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
My correction was developed and tested on 6.1. The other branch is for trunk. So simply merging them is no option.
As my fix solves the problem with rounding taxes, I do not think it a good idea to stall this for a long winding review of all other places where rounding problems might occur, just because somebody did this for trunk.