Tilman Hausherr commented on PDFBOX-4122:

I don't mind, but returning may not always be the correct solution. Sometimes 
throwing an exception or skipping a segment may be better. (This is just a 
general opinion, I didn't look at the individual cases)

> 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