commit 551f8cb28b13d9ca15669773efd8a34550f3e40a
author Hans Boehm <hboehm@google.com> Wed Apr 19 10:24:37 2017 -0700
committer Hans Boehm <hboehm@google.com> Wed Nov 01 14:32:03 2017 -0700
tree 8160ed52304874b184e71c37e67904298fc628f7
parent 9223c18a08481c79f2f363278d3bc47116ae551f

Handle ln(rational multiple of e) better

Bug: 37486067

In particular, ln(e) is now a rational one.

Test: Unit tests, manual test.
Change-Id: I3cc57a4a84ec6667d3c8554c4fc234f9aa7c2e86
(cherry picked from commit bf2e69ae00d7b70562822769c100c3ddcc928b17)

src/com/android/calculator2/UnifiedReal.java

diff --git a/src/com/android/calculator2/UnifiedReal.java b/src/com/android/calculator2/UnifiedReal.java
index f6cf50b..61cac29 100644
--- a/src/com/android/calculator2/UnifiedReal.java
+++ b/src/com/android/calculator2/UnifiedReal.java
@@ -1011,6 +1011,9 @@
     }
 
     public UnifiedReal ln() {
+        if (mCrFactor == CR_E) {
+            return new UnifiedReal(mRatFactor, CR_ONE).ln().add(ONE);
+        }
         if (isComparable(ZERO)) {
             if (signum() <= 0) {
                 throw new ArithmeticException("log(non-positive)");