[ 
https://issues.apache.org/jira/browse/LUCENE-2401?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12858358#action_12858358
 ] 

Earwin Burrfoot commented on LUCENE-2401:
-----------------------------------------

I think at least I will hit that specialized case :)
My filters glue some tokens together. (Die, hard-iterative-API, die!)

> Improve performance of CharTermAttribute(Impl) and also fully implement 
> Appendable
> ----------------------------------------------------------------------------------
>
>                 Key: LUCENE-2401
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2401
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Analysis
>            Reporter: Uwe Schindler
>            Assignee: Uwe Schindler
>             Fix For: 3.1
>
>         Attachments: LUCENE-2401.patch, LUCENE-2401.patch
>
>
> The Appendable.append(CharSequence) method in CharTermAttributes is good for 
> general use. But like StringBuilder has for some common use cases specialized 
> methods, this does the same and adds separate append methods for String, 
> StringBuilder and CharTermAttribute itsself. This methods enable the compiler 
> to directly link the specialized methods and don't use the instanceof checks. 
> The unspecialized method only does the instanceof checks for longer 
> CharSequences (>8 chars), else it simply iterates.
> This patch also fixes the required special "null" handling. append() methods 
> are required by Appendable to append "null", if the argument is null. I dont 
> like this, but its required. Maybe we should document, that we dont dont 
> support it. Otherwise, JDK's formatter fails with formatting null.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to