Hi, I prefer if possible to open a Bugzilla, as this kind of change can be impacting and sometimes you notice issues few months after but don't remember this change. This is particularly needed for Third Party providers.
Regards Philippe On Mon, Dec 22, 2014 at 5:57 PM, sebb <[email protected]> wrote: > On 22 December 2014 at 16:34, Felix Schumacher > <[email protected]> wrote: > > Hi all, > > > > in the converters in org.apache.jmeter.save.converters there are a few > > possible NPE. In the method canConvert(Class) often the implementation > looks > > like > > > > "return arg0.equals(SomeClass.class)" > > > > without arg0 being checked for non-null. > > > > I have a patch, that reverts the logic to > > > > "return SomeClass.class.equals(arg0)" > > > > This is null safe and should return the same values (apart from possible > > NPE). > > > > Can I check those changes in whithout creating a bugzilla entry? > > OK by me. > > Please ensure that the commit only contains these specific changes, > and not any unrelated ones. > Thanks. > > > Regards > > Felix > -- Cordialement. Philippe Mouawad.
