[ 
https://issues.apache.org/jira/browse/PDFBOX-1833?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13915669#comment-13915669
 ] 

Jens Kapitza commented on PDFBOX-1833:
--------------------------------------

just one method refactored. see patch.


pdfbox-eclipse-formatter.xml was updated but there are still confusion at 
formatting for me.

- while control 
- ....

in one file there are multiple conventions
is there a big reformatting patch? (would be great if someone could format the 
code with eclipse rules or whatever)

> BaseParser tidy up
> ------------------
>
>                 Key: PDFBOX-1833
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-1833
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: Parsing
>            Reporter: Jens Kapitza
>            Priority: Minor
>         Attachments: baseparser.patch
>
>   Original Estimate: 0.5h
>  Remaining Estimate: 0.5h
>
> Tidy up logic (should not change the parsing result)
> Character.isWhitespace(c) is the only point wich may have site effects (but i 
> assume there is no File-Seperator in parseCOSHexString)
> so this should pass as it passes befor.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to