Skip to content

Conversation

jimmykane
Copy link

This fixes jslinters reporting warning about null comparison when library is included in frameworks.

null comparison should be strict.

@staabm
Copy link
Member

staabm commented Feb 19, 2015

this breaks a unit tests. It seems 0 or false is also valid in this case.

This could be a bug though.

@jimmykane
Copy link
Author

@staabm Correct. Did not see that. Cancel this please

@staabm staabm closed this Feb 19, 2015
@jzaefferer
Copy link
Collaborator

!= null matches both null and undefined. jshint accepts that via the eqnull option.

@jimmykane
Copy link
Author

That is true.

On Tue Feb 24 2015 at 7:43:36 PM Jörn Zaefferer notifications@github.com
wrote:

!= null matches both null and undefined. jshint accepts that via the
eqnull option.


Reply to this email directly or view it on GitHub
#1406 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants