Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6.0
    • Fix Version/s: 1.7.0
    • Component/s: Parsing
    • Labels:
      None

      Description

      As a precondition of PDFBOX-1199 and other improvements implementing new ways of parsing PDF documents using existing code base it is necessary that subclasses of PDFParser can be created. While this is already possible with the current version accessing some needed methods is prohibited by the 'private' keyword. The attached patch changes this to 'protected'.

      If there are no objections against the patch I will commit it in the next days.

        Activity

        Timo Boehme created issue -
        Hide
        Timo Boehme added a comment -

        added patch which changes some methods from 'private' to 'protected'

        Show
        Timo Boehme added a comment - added patch which changes some methods from 'private' to 'protected'
        Timo Boehme made changes -
        Field Original Value New Value
        Attachment 2012-01-23_PDFParserProtected.patch [ 12511511 ]
        Hide
        Adam Nichols added a comment -

        Looks fine to me

        Show
        Adam Nichols added a comment - Looks fine to me
        Hide
        Mel Martinez added a comment -

        +1

        I am very much in favor of this change. Thanks!

        Show
        Mel Martinez added a comment - +1 I am very much in favor of this change. Thanks!
        Hide
        Timo Boehme added a comment -

        fixed as proposed with svn commit r1298767

        Show
        Timo Boehme added a comment - fixed as proposed with svn commit r1298767
        Timo Boehme made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 1.7.0 [ 12316940 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Timo Boehme
            Reporter:
            Timo Boehme
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development