Skip to content

Commit e029f7c

Browse files
author
Benoit Lagae
committed
Stop reusing xref object nrs & edit tests to fit
DEVSIX-1428
1 parent a3e57f2 commit e029f7c

File tree

3 files changed

+4
-16
lines changed

3 files changed

+4
-16
lines changed

kernel/src/main/java/com/itextpdf/kernel/pdf/PdfXrefTable.java

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -116,20 +116,8 @@ public PdfIndirectReference get(int index) {
116116
* @return created indirect reference.
117117
*/
118118
protected PdfIndirectReference createNextIndirectReference(PdfDocument document) {
119-
PdfIndirectReference reference;
120-
if (freeReferences.size() > 0) {
121-
int num = (int) freeReferences.pollFirst();
122-
reference = xref[num];
123-
if (reference == null) {
124-
reference = new PdfIndirectReference(document, num);
125-
xref[num] = reference;
126-
}
127-
reference.setOffset(0);
128-
reference.clearState(PdfObject.FREE);
129-
} else {
130-
reference = new PdfIndirectReference(document, ++count);
131-
add(reference);
132-
}
119+
PdfIndirectReference reference = new PdfIndirectReference(document, ++count);
120+
add(reference);
133121
return reference.setState(PdfObject.MODIFIED);
134122
}
135123

kernel/src/test/java/com/itextpdf/kernel/pdf/PdfPagesTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ public void randomObjectPagesTest() throws IOException {
171171
PdfPage testPage = document.removePage(1000);
172172
Assert.assertTrue(testPage.getPdfObject().getIndirectReference() == null);
173173
document.addPage(1000, testPage);
174-
Assert.assertTrue(testPage.getPdfObject().getIndirectReference().getObjNumber() < xrefSize);
174+
Assert.assertTrue(testPage.getPdfObject().getIndirectReference().getObjNumber() == xrefSize);
175175

176176
for (int i = 0; i < pages.length; i++) {
177177
Assert.assertEquals("Remove page", true, document.removePage(pages[i]));

kernel/src/test/java/com/itextpdf/kernel/pdf/PdfReaderTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ public void pagesTest01() throws IOException {
375375

376376
Assert.assertTrue(testPage.getPdfObject().getIndirectReference() == null);
377377
document.addPage(1000, testPage);
378-
Assert.assertTrue(testPage.getPdfObject().getIndirectReference().getObjNumber() < xrefSize);
378+
Assert.assertTrue(testPage.getPdfObject().getIndirectReference().getObjNumber() == xrefSize);
379379

380380
for (int i = 1; i < document.getNumberOfPages() + 1; i++) {
381381
PdfPage page = document.getPage(i);

0 commit comments

Comments
 (0)