[ 
https://issues.apache.org/jira/browse/LUCENENET-340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12835307#action_12835307
 ] 

Digy commented on LUCENENET-340:
--------------------------------

+1 for me too. But a new patch is needed for 2.9.2
DIGY

> Fieldable.ReaderValue should return TextReader not StreamReader
> ---------------------------------------------------------------
>
>                 Key: LUCENENET-340
>                 URL: https://issues.apache.org/jira/browse/LUCENENET-340
>             Project: Lucene.Net
>          Issue Type: Bug
>            Reporter: Andy Pook
>            Priority: Minor
>         Attachments: LUCENENET-340-Fieldable.ReaderValue-r911211.patch, 
> LUCENENET-340-Filedable.ReaderValue.patch
>
>
> Using the more abstract type makes creating custome Fieldables easier as they 
> can use other TextReader descendants (such as StringReader).
> Internally the main use seems to be from DocInverterPerField (eg line 125) 
> where it is used via a local member of type TextReader.
> The original Java uses the abstract Reader type. Is there a reason why the 
> dotNet conversion uses StreamReader here while using TextReader elsewhere?

-- 
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