From da9d8ec9b5f0235fd44756f53fca7f0be29e29b3 Mon Sep 17 00:00:00 2001 From: "Owen W. Taylor" Date: Wed, 27 Apr 2011 12:25:20 -0400 Subject: [PATCH] Add missing patch --- ...ff-by-one-when-incrementally-adding-.patch | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 appDisplay-Fix-off-by-one-when-incrementally-adding-.patch diff --git a/appDisplay-Fix-off-by-one-when-incrementally-adding-.patch b/appDisplay-Fix-off-by-one-when-incrementally-adding-.patch new file mode 100644 index 0000000..4329abb --- /dev/null +++ b/appDisplay-Fix-off-by-one-when-incrementally-adding-.patch @@ -0,0 +1,31 @@ +From 4e9ef93c0691dab6d90935353e6de981cbddac54 Mon Sep 17 00:00:00 2001 +From: Owen W. Taylor +Date: Wed, 27 Apr 2011 07:50:10 -0400 +Subject: [PATCH] appDisplay: Fix off-by-one when incrementally adding + application icons + +A "cosmetic" code arrangement I requested in code review resulted +in one too few items being removed from the queue for each incremental +chunk of icons added. Fix. + +https://bugzilla.gnome.org/show_bug.cgi?id=648739 +--- + js/ui/appDisplay.js | 2 +- + 1 files changed, 1 insertions(+), 1 deletions(-) + +diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js +index cd502ef..6fe12a4 100644 +--- a/js/ui/appDisplay.js ++++ b/js/ui/appDisplay.js +@@ -126,7 +126,7 @@ AlphabeticalView.prototype = { + if (currentTimeMillis - startTimeMillis > MAX_APPLICATION_WORK_MILLIS) + break; + } +- this._pendingAppIds.splice(0, i); ++ this._pendingAppIds.splice(0, i + 1); + if (this._pendingAppIds.length > 0) { + return true; + } else { +-- +1.7.4.4 +