commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz" <phil.ste...@gmail.com>
Subject Re: [math] possible issue with ValueServer.getNextReplay()
Date Mon, 14 Jan 2008 05:24:12 GMT
On Jan 13, 2008 1:16 PM, Luc Maisonobe <Luc.Maisonobe@free.fr> wrote:
> Still playing with findbugs, I have encounter another thing I don't
> understand.
>
> The getNextReplay() method in ValueServer has a wrap feature to restart
> file reading when EOF is encountered. This feature is triggered when
> readLine returns null, but only one test is performed.
>
> I think that this  could produce a NullPointerException is the provided
> file is empty. I don't see any check for this. Adding a check would
> imply changing the method signature by adding a new Exception (the
> method only declares throwing IOException).
>
> Any thoughts about this ?

I have not tried with an empty file, but it looks to me like you
should get either IOException or NumberFormatException, as advertised.
  Did you get NPE with an empty file?

Phil

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


Mime
View raw message