Skip to content

Commit 3f297c8

Browse files
committed
Make adoption agency loop condition more explicit
1 parent b2b852f commit 3f297c8

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

lib/parser/index.js

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -890,19 +890,13 @@ function aaObtainFurthestBlock(p, formattingElementEntry) {
890890

891891
//Step 13 of the algorithm
892892
function aaInnerLoop(p, furthestBlock, formattingElement) {
893-
var element = null,
894-
lastElement = furthestBlock,
893+
var lastElement = furthestBlock,
895894
nextElement = p.openElements.getCommonAncestor(furthestBlock);
896895

897-
for (var i = 0; ; i++) {
898-
element = nextElement;
899-
896+
for (var i = 0, element = nextElement; element !== formattingElement; i++, element = nextElement) {
900897
//NOTE: store next element for the next loop iteration (it may be deleted from the stack by step 9.5)
901898
nextElement = p.openElements.getCommonAncestor(element);
902899

903-
if (element === formattingElement)
904-
break;
905-
906900
var elementEntry = p.activeFormattingElements.getElementEntry(element),
907901
counterOverflow = elementEntry && i >= AA_INNER_LOOP_ITER,
908902
shouldRemoveFromOpenElements = !elementEntry || counterOverflow;

0 commit comments

Comments
 (0)