<div dir="ltr">On Mon, Apr 15, 2013 at 8:09 PM, Michal Čihař <span dir="ltr"><<a href="mailto:michal@cihar.com" target="_blank">michal@cihar.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all<br>
<br>
I've took some time to look at jshint checks found on our code:<br>
<br>
<a href="http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/3132/violations/?#jslint" target="_blank">http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/3132/violations/?#jslint</a><br>
<br>
I've fixed some obvious ones (like missing ;, typecast safe comparing,<br>
missing radix for parseInt), but there are still many of them. Most of<br>
them fit into following case:<br>
<br>
<br>
- ['strSave'] is better written in dot notation.<br>
<br>
This error is raised to highlight an unnecessarily verbose and<br>
potentially confusing piece of code. More detailed explanation is at:<br>
<br>
<a href="http://jslinterrors.com/a-is-better-written-in-dot-notation/" target="_blank">http://jslinterrors.com/a-is-better-written-in-dot-notation/</a><br>
<br>
<br>
- 'type' is already defined. / 'type' used out of scope.<br>
<br>
Most of these come from following constructs:<br>
<br>
if (foo) {<br>
    var bar = 1;<br>
} else {<br>
    var bar = 1;<br>
}<br>
alert(bar);<br>
<br>
This is not an issue for javascript (it has no block scope for<br>
variables), it can be confusing, more details at:<br>
<br>
<a href="http://www.jshint.com/docs/#funcscope" target="_blank">http://www.jshint.com/docs/#funcscope</a><br>
<br>
<br>
So both are more just matter of coding style rather than real bug and<br>
can be disabled in jshint. The question is whether we would prefer to<br>
hide these warnings or fix them.<span class="HOEnZb"></span><br></blockquote></div><br></div><div class="gmail_extra">I think we should fix them out. It'll increase our code quality. I'll fix some of them if no objections.<br>
<br clear="all"></div><div class="gmail_extra">Regards !<br></div><div class="gmail_extra">-- <br><font size="4">Chanaka Dharmarathna<br></font><div><font color="#999999"><b>Virtusa (Pvt) Ltd. | </b></font><b style="color:rgb(153,153,153)">Sri Lanka</b></div>

</div></div>