From ef87236cea64b45dbcacfc39bcd1d02b6c72dcf2 Mon Sep 17 00:00:00 2001 From: "bzbarsky%mit.edu" Date: Sun, 5 Oct 2003 23:46:55 +0000 Subject: [PATCH] Useless test for unsigned >= 0. Bug 221292, patch by mats.palmgren@bredband.net (Mats Palmgren), r+sr=bzbarsky --- content/events/src/nsEventStateManager.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/content/events/src/nsEventStateManager.cpp b/content/events/src/nsEventStateManager.cpp index 65b8d467b857..7ddb870d30e6 100644 --- a/content/events/src/nsEventStateManager.cpp +++ b/content/events/src/nsEventStateManager.cpp @@ -3576,9 +3576,11 @@ nsEventStateManager::GetNextTabbableContent(nsIContent* aRootContent, } } } - PRInt32 increment = forward ? 1 : - 1; - PRInt32 start = index < count ? index + increment : (forward ? 0 : count - 1); - for (index = start; index < count && index >= 0; index += increment) { + PRInt32 increment = forward ? 1 : -1; + // In the following two lines we might substract 1 from zero, + // the |index < count| loop condition will be false in that case too. + index = index < count ? index + increment : (forward ? 0 : count - 1); + for (; index < count; index += increment) { //Iterate over the children. childArea = map->GetChildAt(index);