From 92b2a27edf799f1a836f3c39c3e62eb8ef3ab766 Mon Sep 17 00:00:00 2001
From: serso <serso@serso-laptop.(none)>
Date: Sat, 12 Nov 2011 16:18:36 +0400
Subject: [PATCH] android_calculator-7: Bug with displaying result

---
 .../android/view/AutoResizeTextView.java      | 65 +++++++++++++------
 1 file changed, 45 insertions(+), 20 deletions(-)

diff --git a/src/main/java/org/solovyev/android/view/AutoResizeTextView.java b/src/main/java/org/solovyev/android/view/AutoResizeTextView.java
index 24a719ad..317f8f0e 100644
--- a/src/main/java/org/solovyev/android/view/AutoResizeTextView.java
+++ b/src/main/java/org/solovyev/android/view/AutoResizeTextView.java
@@ -8,11 +8,12 @@ package org.solovyev.android.view;
 
 import android.content.Context;
 import android.graphics.Canvas;
+import android.text.Editable;
 import android.text.Layout.Alignment;
 import android.text.StaticLayout;
 import android.text.TextPaint;
 import android.util.AttributeSet;
-import android.util.TypedValue;
+import android.util.Log;
 import android.widget.TextView;
 
 /**
@@ -163,7 +164,9 @@ public class AutoResizeTextView extends TextView {
 	@Override
 	protected void onLayout(boolean changed, int left, int top, int right, int bottom) {
 		if (changed || needsResize) {
-			resizeText(right - left, bottom - top, getText());
+			int widthLimit = (right - left) - getCompoundPaddingLeft() - getCompoundPaddingRight();
+			int heightLimit = (bottom - top) - getCompoundPaddingBottom() - getCompoundPaddingTop();
+			resizeText(widthLimit, heightLimit, getText());
 		}
 		super.onLayout(changed, left, top, right, bottom);
 	}
@@ -175,7 +178,7 @@ public class AutoResizeTextView extends TextView {
 		resizeText(getText());
 	}
 
-	public void resizeText(final CharSequence text) {
+	private void resizeText(final CharSequence text) {
 		int heightLimit = getHeight() - getPaddingBottom() - getPaddingTop();
 		int widthLimit = getWidth() - getPaddingLeft() - getPaddingRight();
 		resizeText(widthLimit, heightLimit, text);
@@ -188,7 +191,8 @@ public class AutoResizeTextView extends TextView {
 	 * @param height
 	 * @param text
 	 */
-	public void resizeText(int width, int height, final CharSequence text) {
+	private void resizeText(int width, int height, CharSequence text) {
+		Log.d(this.getClass().getName(), "Resizing: w=" + width + ", h=" + height + ", text='" + text + "'");
 
 		// Do not resize if the view does not have dimensions or there is no text
 		if (text == null || text.length() == 0 || height <= 0 || width <= 0) {
@@ -200,29 +204,46 @@ public class AutoResizeTextView extends TextView {
 
 		// Store the current text size
 		float oldTextSize = textPaint.getTextSize();
+		Log.d(this.getClass().getName(), "Old text size: " + oldTextSize);
 
 		// If there is a max text size set, use the lesser of that and the default text size
 		float newTextSize = 100;
 
-		// Get the required text height
-		int newTextHeight = getTextRect(text, textPaint, width, newTextSize);
+		int newTextHeight;
 
-		if (newTextHeight > height) {
-			// Until we either fit within our text view or we had reached our min text size, incrementally try smaller sizes
-			while (newTextHeight > height) {
-				if (newTextSize <= minTextSize) {
-					break;
+		if (text instanceof Editable) {
+			((Editable) text).append("|");
+		}
+
+		try {
+
+			// Get the required text height
+			newTextHeight = getTextRect(text, textPaint, width, newTextSize);
+
+			logDimensions(newTextSize, newTextHeight);
+			if (newTextHeight > height) {
+				// Until we either fit within our text view or we had reached our min text size, incrementally try smaller sizes
+				while (newTextHeight > height) {
+					if (newTextSize <= minTextSize) {
+						break;
+					}
+					newTextSize = Math.max(newTextSize - 2, minTextSize);
+					newTextHeight = getTextRect(text, textPaint, width, newTextSize);
+					logDimensions(newTextSize, newTextHeight);
+				}
+			} else {
+				while (newTextHeight < height) {
+					if (newTextSize <= minTextSize) {
+						break;
+					}
+					newTextSize = Math.max(newTextSize + 2, minTextSize);
+					newTextHeight = getTextRect(text, textPaint, width, newTextSize);
+					logDimensions(newTextSize, newTextHeight);
 				}
-				newTextSize = Math.max(newTextSize - 2, minTextSize);
-				newTextHeight = getTextRect(text, textPaint, width, newTextSize);
 			}
-		} else {
-			while (newTextHeight < height) {
-				if (newTextSize <= minTextSize) {
-					break;
-				}
-				newTextSize = Math.max(newTextSize + 2, minTextSize);
-				newTextHeight = getTextRect(text, textPaint, width, newTextSize);
+		} finally {
+			if (text instanceof Editable) {
+				((Editable) text).delete(text.length() - 1, text.length());
 			}
 		}
 
@@ -259,6 +280,10 @@ public class AutoResizeTextView extends TextView {
 		needsResize = false;
 	}
 
+	private void logDimensions(float newTextSize, int newTextHeight) {
+		Log.d(this.getClass().getName(), "Nex text size: " + newTextSize + ", new text height: " + newTextHeight);
+	}
+
 	// Set the text size of the text paint object and use a static layout to render text off screen before measuring
 	private int getTextRect(CharSequence source, TextPaint paint, int width, float textSize) {
 		// Update the text paint object