tika-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arjohn Kampman (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TIKA-782) Add support for parsing binary data in RTF files
Date Thu, 17 Nov 2011 14:34:52 GMT

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

Arjohn Kampman commented on TIKA-782:
-------------------------------------

I'll make the necessary changes.

Do you mind if I changed the <long> parameter of processControlWord() to an <int>.There's
a comment above that method that says:

    // Param is long because spec says max value is 1+ Integer.MAX_VALUE!

However, Microsoft's RTF 1.9.1 specs says:

    The range of the values for the number is nominally -32768 through 32767,
    i.e., a signed 16-bit integer. A small number of control words take values
    in the range‌ −2,147,483,648 to 2,147,483,647 (32-bit signed integer).

This is exactly the range of Java's int. Not sure which spec the comment is referring to though.

                
> Add support for parsing binary data in RTF files
> ------------------------------------------------
>
>                 Key: TIKA-782
>                 URL: https://issues.apache.org/jira/browse/TIKA-782
>             Project: Tika
>          Issue Type: Improvement
>          Components: parser
>    Affects Versions: 1.0
>            Reporter: Arjohn Kampman
>            Assignee: Michael McCandless
>         Attachments: bin.patch, bin2.patch
>
>
> The current RTF parser doesn't process \bin control words yet. These control words are
followed by a specific amount of binary data. Because of this, the RTF parser trips over some
of these bytes in a number of (classified) documents.
> I've implemented processing of the \bin control word, but it required of the core parsing
algorithm. IMHO, it also improved readability of the code. I hope you will accept this patch.
Please let me know if the patch requires modifications.
> Apart from the \bin code word, this patch also makes the parser stop after reading the
document-closing '}' character. In a number of files (again, classified), the parser would
include non-readable characters that appeared after this closing brace.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Mime
View raw message