From 3497e9d84d861fc9083c6b1fe2b57d7e1e6b9e7d Mon Sep 17 00:00:00 2001 From: Tim Su Date: Mon, 22 Aug 2011 23:23:12 -0700 Subject: [PATCH] Minor tweak to edit people control set to wrap dialog dismissal so it doesn't leak. --- .../com/todoroo/astrid/actfm/EditPeopleControlSet.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java b/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java index 3b0cf17dd..04d1f23b3 100644 --- a/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java +++ b/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java @@ -509,9 +509,9 @@ public class EditPeopleControlSet implements TaskEditControlSet { Intent broadcastIntent = new Intent(AstridApiConstants.BROADCAST_EVENT_REFRESH); ContextManager.getContext().sendBroadcast(broadcastIntent, AstridApiConstants.PERMISSION_READ); + DialogUtilities.dismissDialog(activity, pd); activity.runOnUiThread(new Runnable() { public void run() { - DialogUtilities.dismissDialog(activity, pd); showSaveToast(); activity.finish(); } @@ -525,11 +525,7 @@ public class EditPeopleControlSet implements TaskEditControlSet { activity.getString(R.string.SyP_ioerror), android.R.drawable.ic_dialog_alert, e.toString(), null); } finally { - activity.runOnUiThread(new Runnable() { - public void run() { - DialogUtilities.dismissDialog(activity, pd); - } - }); + DialogUtilities.dismissDialog(activity, pd); } }