tonytomov/jqGrid — pull request #464 by OlegKi (2013-05-21 13:34:06+00:00)
Title: minor fixes suggested by LSLint

Hello Tony,

I included in the pull request only clear places of the code which fond by JSLint. Mostly it's usage of `===` or `!==` instead of `==` and `!=` with **strings** or **number** on both sides.

Another thing about cosmetic changes in the code: there are two standard places where variables need be defined, but not used:

1) there are typical code like

try { ... } catch(e){}

where variable `e` will be not used. JSLint now suggest to use the name `ignore` in the case. The name seems me personally good choice because it's more readable.

2) definition of a variable of callback function (like event of in `$.each`) like

$.each(data,function(i,v){
// i will be not used inside of the function, but v will be used
}

What do you think about renaming of such unused variable `i` to the name `unused` or `unused1`, `unused2`?

I don't included such changed in the pull request. I want just know your opinion about such changes.

Best regards
Oleg