I've found this saying URLs are not good as keys for map/set (not performance issues though): http://alan.blog-city.com/do_not_use_javaneturl_as_a_key_in_hashmap.htm
Author advices to use URL.toString() as key instead. As for the false positives, I agree completely, obviously most of those ~10k defects are false positives. This shouldn't be taken too seriously. 2011/8/1 Michael Gentry <[email protected]> > I'll have to look more later, but these tools often generate lots of > false positives and need a way to silence warnings from run-to-run. > > mrg > > > On Mon, Aug 1, 2011 at 8:42 AM, Andrus Adamchik <[email protected]> > wrote: > > Depends on whether their assertion about DNS lookups is true. We are > using local URLs anyways, and nobody complained to date, but certainly worth > a run in a debugger. > > > > On Aug 1, 2011, at 3:30 PM, Michael Gentry wrote: > > > >> Is this really a blocker? > >> > >> > https://analysis.apache.org/drilldown/violations/org.apache.cayenne:cayenne-parent?priority=BLOCKER# > >> > >> Thanks, > >> > >> mrg > >> > >> > >> On Mon, Aug 1, 2011 at 6:04 AM, Aristedes Maniatis <[email protected]> > wrote: > >>> https://analysis.apache.org/dashboard/index/67484 > >>> > >>> We can certainly throw out all the findbugs/clover/etc cruft now. This > is > >>> much better. > >>> > >>> Ari > >>> > >>> > >>> -- > >>> --------------------------> > >>> Aristedes Maniatis > >>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A > >>> > >> > > > > > -- Andrey
