[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [incquery-dev] ProjectGenerationHelper.removeAllExtension
|
Hi,
good catch. What I have meant was to check for their opposite. :) Lets hope its better now (http://git.eclipse.org/c/incquery/org.eclipse.incquery.git/commit/?id=d6c50ccc32ad31da43599ff878b75ad89a7d4361).
Cheers,
Zoli
-- Zoltán Ujhelyi
https://www.inf.mit.bme.hu/en/members/ujhelyiz
Fault Tolerant Systems Research Group
Budapest University of Technology and Economics
On 2013.10.22., at 9:55, Tamas Szabo <tamas.szabo@xxxxxxxxx> wrote:
> ProjectGenerationHelper.removeAllExtension lines 665, 666:
> What is the reason behind the implementation? First we check whether the StringExtensions.isNullOrEmpty(extensionId)
> holds (which is true for a null extensionId) and then we invoke a method on the null value. I have just found this thing
> due to several NPEs after a full clean was initiated.
>
> --
> Tamas Szabo
> Software engineer
>
> Tel.: +49 711 342 191 0
> Fax.: +49 711 342 191 29
> Mobil: +49 171 565 416 9
> Web: www.itemis.de
> Mail: tamas.szabo@xxxxxxxxx
> Skype: szabta89
>
> itemis AG
> Niederlassung Süd
> Meitnerstr. 10
> 70563 Stuttgart
>
> Rechtlicher Hinweis:
> Registergericht: Amtsgericht Dortmund HRB 20621 | Sitz der Gesellschaft:
> Lünen
> Vorstand: Jens Wagener (Vorsitzender) | Wolfgang Neuhaus | Dr. Georg
> Pietrek | Jens Trompeter | Sebastian Neus
> Aufsichtsrat: Prof. Dr. Burkhard Igel (Vorsitzender) | Stephan Grollmann
> | Michael Neuhaus
>
> _______________________________________________
> incquery-dev mailing list
> incquery-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/incquery-dev
