Do row adding in bulk

guustysebie · guustysebie · commit 0626cd422a27 · 2025-01-20T18:59:58.000Z
DEVSIX-8836
diff --git a/layout/src/main/java/com/itextpdf/layout/tagging/TableTaggingRule.java b/layout/src/main/java/com/itextpdf/layout/tagging/TableTaggingRule.java
@@ -86,6 +86,7 @@ public boolean onTagFinish(LayoutTaggingHelper taggingHelper, TaggingHintKey tab
                 taggingHelper.moveKidHint(nonCellKid, tableHintKey);
             }
         }
+        List<TaggingDummyElement> rows = new ArrayList<>();
         for (TreeMap<Integer, TaggingHintKey> rowTags : tableTags.values()) {
             TaggingDummyElement row = new TaggingDummyElement(StandardRoles.TR);
             TaggingHintKey rowTagHint = LayoutTaggingHelper.getOrCreateHintKey(row);
@@ -98,8 +99,9 @@ public boolean onTagFinish(LayoutTaggingHelper taggingHelper, TaggingHintKey tab
                 }
                 tableCellTagsUnindexed = null;
             }
-            taggingHelper.addKidsHint(tbodyTag, Collections.<TaggingDummyElement>singletonList(row), -1);
+            rows.add(row);
         }
+        taggingHelper.addKidsHint(tbodyTag, rows, -1);
 
         for (TaggingHintKey nonCellKid : nonCellKids) {
             if (StandardRoles.CAPTION.equals(getKidRole(nonCellKid,taggingHelper))) {

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,7 @@ public boolean onTagFinish(LayoutTaggingHelper taggingHelper, TaggingHintKey tab`
`86`	`86`	`taggingHelper.moveKidHint(nonCellKid, tableHintKey);`
`87`	`87`	`}`
`88`	`88`	`}`
	`89`	`+ List<TaggingDummyElement> rows = new ArrayList<>();`
`89`	`90`	`for (TreeMap<Integer, TaggingHintKey> rowTags : tableTags.values()) {`
`90`	`91`	`TaggingDummyElement row = new TaggingDummyElement(StandardRoles.TR);`
`91`	`92`	`TaggingHintKey rowTagHint = LayoutTaggingHelper.getOrCreateHintKey(row);`
`@@ -98,8 +99,9 @@ public boolean onTagFinish(LayoutTaggingHelper taggingHelper, TaggingHintKey tab`
`98`	`99`	`}`
`99`	`100`	`tableCellTagsUnindexed = null;`
`100`	`101`	`}`
`101`		`- taggingHelper.addKidsHint(tbodyTag, Collections.<TaggingDummyElement>singletonList(row), -1);`
	`102`	`+ rows.add(row);`
`102`	`103`	`}`
	`104`	`+ taggingHelper.addKidsHint(tbodyTag, rows, -1);`
`103`	`105`
`104`	`106`	`for (TaggingHintKey nonCellKid : nonCellKids) {`
`105`	`107`	`if (StandardRoles.CAPTION.equals(getKidRole(nonCellKid,taggingHelper))) {`