Correctly track Activity in Evaluator
Bug: 33703223
Evaluator was remembering the Activity with which it had been
created, not necessarily the current one. This interfered with
proper display of things like timeout messages that rely on the
remembered Activity.
Use the Application rather than Activity context for the DB helper,
since it outlives the Activity.
Remove unused mActivity from ExpressionDB.
Also acouple of trivial cleanups for minor annoyances found while
debugging:
1. Add some missing final declarations.
2. Fix a comment.
Change-Id: Iefe1fb588f66a1c77c82164680306377917c07af
diff --git a/src/com/android/calculator2/CalculatorExpr.java b/src/com/android/calculator2/CalculatorExpr.java
index 321ba5f..18bb6cf 100644
--- a/src/com/android/calculator2/CalculatorExpr.java
+++ b/src/com/android/calculator2/CalculatorExpr.java
@@ -977,7 +977,7 @@
* Is the current expression worth evaluating?
*/
public boolean hasInterestingOps() {
- int last = trailingBinaryOpsStart();
+ final int last = trailingBinaryOpsStart();
int first = 0;
if (last > first && isOperatorUnchecked(first, R.id.op_sub)) {
// Leading minus is not by itself interesting.