lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Digy (JIRA)" <j...@apache.org>
Subject [jira] Commented: (LUCENENET-340) Fieldable.ReaderValue should return TextReader not StreamReader
Date Thu, 18 Feb 2010 17:58:27 GMT

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


Mime
View raw message