Skip to content

Commit 78d7d91

Browse files
committed
Mark internal classes of PdfDictionaryEntrySet and PdfDictionaryValues as static
.net porting
1 parent 18c12cc commit 78d7d91

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

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

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public boolean remove(Object o) {
6868

6969
@Override
7070
public Iterator<Map.Entry<PdfName, PdfObject>> iterator() {
71-
return new DirectIterator();
71+
return new DirectIterator(set.iterator());
7272
}
7373

7474
@Override
@@ -81,8 +81,12 @@ public void clear() {
8181
set.clear();
8282
}
8383

84-
private class DirectIterator implements Iterator<Map.Entry<PdfName, PdfObject>> {
85-
Iterator<Map.Entry<PdfName, PdfObject>> parentIterator = set.iterator();
84+
private static class DirectIterator implements Iterator<Map.Entry<PdfName, PdfObject>> {
85+
Iterator<Map.Entry<PdfName, PdfObject>> parentIterator;
86+
87+
public DirectIterator(Iterator<Map.Entry<PdfName, PdfObject>> parentIterator) {
88+
this.parentIterator = parentIterator;
89+
}
8690

8791
@Override
8892
public boolean hasNext() {
@@ -100,11 +104,11 @@ public void remove() {
100104
}
101105
}
102106

103-
private class DirectEntry implements Map.Entry<PdfName, PdfObject> {
107+
private static class DirectEntry implements Map.Entry<PdfName, PdfObject> {
104108

105109
Map.Entry<PdfName, PdfObject> entry;
106110

107-
public DirectEntry(Map.Entry<PdfName, PdfObject> entry) {
111+
DirectEntry(Map.Entry<PdfName, PdfObject> entry) {
108112
this.entry = entry;
109113
}
110114

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

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,11 +101,15 @@ public void clear() {
101101

102102
@Override
103103
public Iterator<PdfObject> iterator() {
104-
return new DirectIterator();
104+
return new DirectIterator(collection.iterator());
105105
}
106106

107-
private class DirectIterator implements Iterator<PdfObject> {
108-
Iterator<PdfObject> parentIterator = collection.iterator();
107+
private static class DirectIterator implements Iterator<PdfObject> {
108+
Iterator<PdfObject> parentIterator;
109+
110+
DirectIterator(Iterator<PdfObject> parentIterator) {
111+
this.parentIterator = parentIterator;
112+
}
109113

110114
@Override
111115
public boolean hasNext() {

0 commit comments

Comments
 (0)