tonytomov/jqGrid — comment on commit 617a56648e37423d69ceaf5c5a63f16130d05181
File:  line 0
By OlegKi:

It's not really important which tool will be used. Only the suggestions are important. Of course the changes of the line

o.url = o.url ? o.url : $t.p.editurl;

and the line

tmp = sucret[1] ? sucret[1] : tmp;

to

o.url = o.url || $t.p.editurl;

and

tmp = sucret[1] || tmp;

do improve the code a little. Moreover JSLint can gives another practical recommendations. For example information about unused variables like `i,n` from [the line 356](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L356):

$.each($t.p.colModel, function(i,n){

unused `i` in the [line 388](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L388):

$($t.p.colModel).each(function(i){

unused `e` in the [line 465](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L465), [line 482](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L482), [line 502](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L502), [line 529](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L529)

onClickButton : function ( e ) {

I find the suggestions good.

Best regards
Oleg