OK, removed it in https://svn.apache.org/r1619785
Robert On Fri, Aug 22, 2014 at 9:52 AM, Carsten Ziegeler <[email protected]> wrote: > It's not used - it's there for an outstanding but questionable issue > I guess we can remove it for now - it's in the scm anyway > > Carsten > > > 2014-08-21 22:39 GMT+02:00 Robert Munteanu <[email protected]>: > >> Hi, >> >> Browsing the list of open Sonar issues I found one [1] which would >> always cause a NPE in >> org.apache.sling.installer.core.impl.tasks.RefreshOptionalPackagesTask >> ( packageAdmin is always null ). >> >> I'm not familiar with that area of the code, but by looking at the >> code it seems that this class is never used. Is that the case? If so, >> can it be deleted? >> >> Thanks, >> >> Robert >> >> >> [1]: >> https://analysis.apache.org/drilldown/issues/35893?&rule=findbugs%3ANP_ALWAYS_NULL&rule_sev=CRITICAL&severity=CRITICAL# >> > > > > -- > Carsten Ziegeler > Adobe Research Switzerland > [email protected] -- Sent from my (old) computer
