Re: Inconsistent String Comparisons

[Michael Brohl]

+1

Regards,

Michael

Am 09.01.17 um 10:27 schrieb Nicolas Malin:

Hello Devanshu,

Yes your welcome to submit a patch :)

Nicolas

Le 09/01/2017 à 10:22, Devanshu Vyas a écrit :

Hello Devs,

I found an inconsistency in the code for string comparison
*statusId.equals("PRUN_COMPLETED")* whereas it should be written as
*"PRUN_COMPLETED".equals(statusId)*

NULL.


this I can provide a patch for correcting code.

Let me know your thoughts.

Thanks & Regards,
Devanshu Vyas.

cause the former can throw NullPointerException if the variable found to be This code pattern can be found at several places and if you all agree with

smime.p7s
Description: S/MIME Cryptographic Signature