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

Jukka Zitting commented on TIKA-58:
-----------------------------------

+1 Nice

Some comments:

* I would rather keep the class named as HtmlParser instead of NekoHtmlParser 
to avoid exposing implementation details.

* Use spaces instead of tabs for indentation.

* Does anyone know if NekoHTML is actively maintained, or if the idea of 
adopting it in Xerces is still being considered?


> Replace jtidy html parser with nekohtml based parser
> ----------------------------------------------------
>
>                 Key: TIKA-58
>                 URL: https://issues.apache.org/jira/browse/TIKA-58
>             Project: Tika
>          Issue Type: Improvement
>          Components: general
>            Reporter: Sami Siren
>            Assignee: Sami Siren
>            Priority: Minor
>         Attachments: TIKA-58.diff
>
>
> Following patch will replace the JTidy based html parser with NekoHTML based 
> sax parser. It only provides the same functionality that the JTidy based one 
> (extracts a title into metadata) and passes other sax events through. Speed 
> improvement is around 100%.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to