valerybokov edited a comment on pull request #107: URL: https://github.com/apache/pdfbox/pull/107#issuecomment-890298627
Hi, @THausherr! 1 Maybe this is not a problem and it will not be. I found that COSArray.setString can set null as a value. Some methods, such as COSArray.indexOf, use an objects collection with no null checks. 2 PDAnnotation.getBorder. If border.size() >= 3 don't need to "create a copy to avoid changing the PDF"? 3 static final fields of PDRectangle are modifiable (LETTER, LEGAL etc.) 4 TrailerValidationProcess.checkTrailersForLinearizedPDF14. Row 96. A last variable can be null. 5 PreflightConfiguration.SHADING_PATTERN_PROCESS. Typo in value. Double 'd'. 6 RubberStampAnnotationValidator. Maybe wrong comment: BudderStampAnnotation or RubberStampAnnotation 7 class Decrypt. Wrong command name (row 41: "decrpyt") 8 DomXmpParser.manageDefinedType (382, 394, 419). A parseLiDescription method able to return null. Can ast variable be null here. 9 TiffSchema.IMAGE_LENGHT instead of IMAGE_LENGTH I remember about preflight but these notes were written before information about preflight removing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
