pivot-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sandro Martini <sandro.mart...@gmail.com>
Subject Re: Fwd: Small things by FindBugs
Date Mon, 06 Sep 2010 12:53:02 GMT

Hi Greg,
thanks for the fixes ...

> All of these are because DisplayHost is serializable by inheritance -
> however, no one should be serializing it. You can ignore them.
maybe we can try to make it transient ... and if this doesn't solve I agree
that we can keep it as is.

And, sorry these are still to check (are all trivial):

Write to static field
org.apache.pivot.tutorials.webqueries.Expenses.instance from instance method
new org.apache.pivot.tutorials.webqueries.Expenses()
  -- I've seen that the static field is not read, so maybe could be removed

org.apache.pivot.tests.sql.ResultListTest.main(String[]) may fail to close
  -- just add a nested try/finally also for the close

ignores exceptional return value of java.io.File.delete()
  -- just catch also missing exceptions ...

I can fix them this evening, tell me if I have to do ...


View this message in context: http://apache-pivot-developers.417237.n3.nabble.com/Small-things-by-FindBugs-tp1415684p1426083.html
Sent from the Apache Pivot - Developers mailing list archive at Nabble.com.

View raw message