Skip to content

Commit 2cf618d

Browse files
ar3emiText-CI
authored andcommitted
Remove TODOs in Lab and IccBased colors
It seems that these TODOs were added in accordance with the note about the Range entry in Table 65 of the PDF specification 32000-1: 2008. But it seems that this note is relevant to readers. Thus, we may not take this fact into account when writing to a PDF file. DEVSIX-4013
1 parent e7a1b1f commit 2cf618d

File tree

3 files changed

+1
-2
lines changed

3 files changed

+1
-2
lines changed

kernel/src/main/java/com/itextpdf/kernel/colors/IccBased.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ public class IccBased extends Color {
5353
private static final long serialVersionUID = -2204252409856288615L;
5454

5555
public IccBased(PdfCieBasedCs.IccBased cs) {
56-
// TODO if zero is outside of the Range, default value should be the nearest to the zero valid value
5756
this(cs, new float[cs.getNumberOfComponents()]);
5857
}
5958

kernel/src/main/java/com/itextpdf/kernel/colors/Lab.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ public class Lab extends Color {
5050
private static final long serialVersionUID = -103738025280259190L;
5151

5252
public Lab(PdfCieBasedCs.Lab cs) {
53-
// TODO if zero is outside of the Range, default value should be the nearest to the zero valid value
5453
this(cs, new float[cs.getNumberOfComponents()]);
5554
}
5655

kernel/src/main/java/com/itextpdf/kernel/pdf/colorspace/PdfCieBasedCs.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,7 @@ public IccBased(PdfArray pdfObject) {
206206
}
207207

208208
public IccBased(final InputStream iccStream) {
209+
// TODO DEVSIX-4217 add parsing of the Range
209210
this(getInitialPdfArray(iccStream, null));
210211
}
211212

0 commit comments

Comments
 (0)