#543 EqualsAvoidNull should also check for equalsIgnoreCase

closed
Check (274)
5
2012-10-10
2010-09-12
No

Hello,
At the moment, EqualsAvoidNull forces to use "literal".equals(variable) instead of variable.equals("literal"), since variable can be null.

But exactly the same problem may occur when one uses equalsIgnoreCase, so it seems reasonable to include equalsIgnoreCase usage in this check.

Regards,

Ivan

Discussion

  • ktcoder84

    ktcoder84 - 2010-10-16

    This just got brought up recently on our development team as well. I was surprised I'd never noticed it before :) This would be a great enhancement.

    Thanks,
    -Kristen

     
  • Oliver Burn

    Oliver Burn - 2011-01-01

    implemented in release 5.4

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks