[prev in list] [next in list] [prev in thread] [next in thread] 

List:       lucene-dev
Subject:    Re: svn commit: r164695 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/search/Hit.j
From:       Erik Hatcher <erik () ehatchersolutions ! com>
Date:       2005-04-26 19:09:05
Message-ID: 745a9b4a7a23cc308e0851c69b873131 () ehatchersolutions ! com
[Download RAW message or body]

On Apr 26, 2005, at 2:38 PM, Daniel Naber wrote:
> On Tuesday 26 April 2005 02:21, ehatcher@apache.org wrote:
>
>> +  public String toString() {
>> +    try {
>> +      return getDocument().toString();
>> +    } catch (IOException e) {
>> +      return null;
>> +    }
>> +  }
>
> Wouldn't it be better here to re-throw the exception as a 
> RuntimeException?

I don't know.... would it?  I have no preference, though it seems ok to 
me to simply return null since this is the toString method.  For a 
Document, the toString is only useful for debugging anyway.

	Erik


---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org


[prev in list] [next in list] [prev in thread] [next in thread]