Padding DEFAULT_TEXT_PADDING and NO_PADDING null

GlenKPeterson / PdfLayoutManager

Adds line-breaking, page-breaking, tables, and styles to PDFBox

45 stars 20 forks source link

Padding DEFAULT_TEXT_PADDING and NO_PADDING null #8

Closed enm260 closed 9 years ago

enm260 commented 9 years ago

DEFAULT_TEXT_PADDING and NO_PADDING are always null. Suggest adding not null checks to the of() methods before returning these static values.

GlenKPeterson commented 9 years ago

Thanks for reporting this. I'll try to look at it this weekend, or you can submit a patch and I'll look at that.

GlenKPeterson commented 9 years ago

This issue is fixed. Please upgrade to 0.3.3 to use the patched version. I appreciate your patch, but I ended up using a different fix, writing a few unit tests, etc. Thank you very much for reporting this issue!