gnucash maint: Multiple changes pushed

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

gnucash maint: Multiple changes pushed

John Ralls-4
Updated via  https://github.com/Gnucash/gnucash/commit/61cd7999 (commit)
         via  https://github.com/Gnucash/gnucash/commit/536606a8 (commit)
        from  https://github.com/Gnucash/gnucash/commit/e57d4278 (commit)



commit 61cd7999f34a3976fd45bf4668d7012ab3ca4835
Author: John Ralls <[hidden email]>
Date:   Tue Nov 27 22:11:36 2018 +0900

    Fix half-up, half-down, and banker's rounding for negative numbers.
   
    We need to compare the magnitudes of the remainder and the denominator
    in order to round negative numbers correctly. Note that while gnc_numeric
    is constrained to a positive denominator the C++ rounding functions cannot
    assume that constraint in all cases.
   
    Combined with the previous commit, this fixes
    Bug 796949 - Incorrect conversion of 0,01 USD to EUR

commit 536606a89c5b53ad8ef91a63a021b8e564deb3aa
Author: John Ralls <[hidden email]>
Date:   Tue Nov 27 21:25:29 2018 +0900

    Fix extract_common_prices logic.
   
    So that the returned price tuple has the two commodities of interest
    converted to a common currency. Before the first pair that that shared
    any random currency would be returned, perhaps creating an absurd result.



Summary of changes:
 libgnucash/engine/gnc-pricedb.c             | 11 +++++---
 libgnucash/engine/gnc-rational-rounding.hpp | 41 ++++++++++++++++++++++++++---
 2 files changed, 45 insertions(+), 7 deletions(-)

_______________________________________________
gnucash-patches mailing list
[hidden email]
https://lists.gnucash.org/mailman/listinfo/gnucash-patches