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 <sebbaz@gmail.com> wrote:
> On 22 December 2014 at 16:34, Felix Schumacher
> <felix.schumacher@internetallee.de> 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.
|