Skip to content

Commit a878839

Browse files
ars18wrwiText-CI
authored andcommitted
Remove outdated TODOs in TableRenderer
childRenderers are already cleared at the beginning of #layout() currentBorder is considered only for collapsed borders DEVSIX-4164
1 parent 659fc60 commit a878839

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

layout/src/main/java/com/itextpdf/layout/renderer/TableRenderer.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1240,8 +1240,6 @@ protected TableRenderer createSplitRenderer(Table.RowRange rowRange) {
12401240
splitRenderer.rowRange = rowRange;
12411241
splitRenderer.parent = parent;
12421242
splitRenderer.modelElement = modelElement;
1243-
// TODO childRenderers will be populated twice during the relayout.
1244-
// We should probably clean them before #layout().
12451243
splitRenderer.childRenderers = childRenderers;
12461244
splitRenderer.addAllProperties(getOwnProperties());
12471245
splitRenderer.headerRenderer = headerRenderer;
@@ -1545,7 +1543,7 @@ private void correctLayoutedCellsOccupiedAreas(LayoutResult[] splits, int row, i
15451543
int finish = bordersHandler.getFinishRow();
15461544
bordersHandler.setFinishRow(rowRange.getFinishRow());
15471545

1548-
// TODO Correct for collapsed borders only
1546+
// It's width will be considered only for collapsed borders
15491547
Border currentBorder = bordersHandler.getWidestHorizontalBorder(finish + 1);
15501548
bordersHandler.setFinishRow(finish);
15511549
if (skip) {

0 commit comments

Comments
 (0)