Maruan Sahyoun commented on PDFBOX-4122:

That was proposed to to not be more restrictive than today but to provide the 
information. {{COSWriter}} will likely fail mit an NPE if {{null} is being 
passed to public or protected methods (did test some of them). It doesn't 
happen today because the correct input is ensured upfront.

We can of course start to be more restrictive and check the input before moving 
on. IMHO that's better than likely failing.

> Resolve potential NPEs flagged by Sonar
> ---------------------------------------
>                 Key: PDFBOX-4122
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-4122
>             Project: PDFBox
>          Issue Type: Sub-task
>    Affects Versions: 3.0.0 PDFBox
>            Reporter: Maruan Sahyoun
>            Assignee: Maruan Sahyoun
>            Priority: Major
>             Fix For: 3.0.0 PDFBox
> We have many potential NPEs resolved by now but there are still some flagged 
> by Sonar
> https://builds.apache.org/analysis/component_issues?id=org.apache.pdfbox%3Apdfbox-reactor#resolved=false|types=BUG|severities=BLOCKER.
> Although it's very unlikely that we hit them in practice e.g. as this would 
> mean a very corrupt font file I'll try to resolve these debug logging the 
> condition and returning in that case.
> [~tilman] [~lehmi] please comment/revert if you are not happy with the 
> approach. 

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: dev-unsubscr...@pdfbox.apache.org
For additional commands, e-mail: dev-h...@pdfbox.apache.org

Reply via email to