<div dir="ltr"><br><div><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 25, 2013 at 11:01 AM, Keval Sheth <span dir="ltr"><<a href="mailto:krs435@students.poly.edu" target="_blank">krs435@students.poly.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div>
<div style="direction:ltr;font-size:10pt;font-family:Tahoma">Hello Aashish,<br>
<br>
I have made changes in Geo library files( 'fix-jslint-errors' branch), you can check it if the files in Geo directory is passed or not. Also in 'map.js' and 'layer.js' files i am getting below errors:<br>
<br>
<cite>Strict violation.</cite>
<pre> currentElement = this;</pre></div></div></blockquote><div>It's because 'this' was being used in a global context when the function is initially defined. Defining the function anonymously in the assignment call, instead of predefining the function resolves the error.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div style="direction:ltr;font-size:10pt;font-family:Tahoma">
<cite>Unexpected 'this'.</cite>
<pre> this.activeLayer = function() {</pre></div></div></blockquote><div>There is a missing semi-colon on the previous function definition. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div><div style="direction:ltr;font-size:10pt;font-family:Tahoma"><pre><br>Could you please assist us why are we getting this error. When i searched for it i found that we need to add <code><span>/*jslint validthis: true */, but it didn't resolved the error. Could you please assist us with the above two errors?<br>
<br>Regards,<br>Keval <br></span></code></pre>
<br>
<div style="font-size:16px;font-family:'Times New Roman'">
<hr>
<div style="direction:ltr"><font color="#000000" face="Tahoma"><b>From:</b> <a href="mailto:opengeoscience-developers-bounces@public.kitware.com" target="_blank">opengeoscience-developers-bounces@public.kitware.com</a> [<a href="mailto:opengeoscience-developers-bounces@public.kitware.com" target="_blank">opengeoscience-developers-bounces@public.kitware.com</a>] on behalf of Keval Sheth [<a href="mailto:krs435@students.poly.edu" target="_blank">krs435@students.poly.edu</a>]<br>
<b>Sent:</b> Thursday, April 25, 2013 4:57 AM<br>
<b>To:</b> Aashish Chaudhary<div><div class="h5"><br>
<b>Cc:</b> opengeoscience-developers<br>
<b>Subject:</b> Re: [Opengeoscience-developers] js-lint branch<br>
</div></div></font><br>
</div><div><div class="h5">
<div></div>
<div>
<div style="direction:ltr;font-size:10pt;font-family:Tahoma">Hello Aashish,
<div><br>
<div>Thanks a lot. Ya sure, we will handle the geo library once and will be pushing it so that we can find out whether it has been passed it or not.</div>
<div><br>
</div>
<div>Regards,</div>
<div>Keval<br>
<div style="font-size:16px;font-family:'Times New Roman'">
<hr>
<div style="direction:ltr"><font color="#000000" face="Tahoma"><b>From:</b> Aashish Chaudhary [<a href="mailto:aashish.chaudhary@kitware.com" target="_blank">aashish.chaudhary@kitware.com</a>]<br>
<b>Sent:</b> Thursday, April 25, 2013 4:53 AM<br>
<b>To:</b> Keval Sheth<br>
<b>Cc:</b> opengeoscience-developers<br>
<b>Subject:</b> Re: [Opengeoscience-developers] js-lint branch<br>
</font><br>
</div>
<div></div>
<div>
<div dir="ltr">Hi Keval,
<div><br>
</div>
<div>before that, I tried the branch and found that it broke the code. Actually I am thinking that if you and Manas can handle the geo library first (since hopefully no one is touching that code at this moment). And lets have those files passed. If
we run into some issues with those files then we can look into it further. I think if we go module by module or files by files, it will make it easy for us to get it done. </div>
<div><br>
</div>
<div>Does it make sense? </div>
<div><br>
</div>
<div>Thanks,</div>
<div>Aashish</div>
<div><br>
</div>
</div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">On Thu, Apr 25, 2013 at 12:25 AM, Keval Sheth <span dir="ltr">
<<a href="mailto:krs435@students.poly.edu" target="_blank">krs435@students.poly.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div>
<div style="direction:ltr;font-size:10pt;font-family:Tahoma">Hello Aashish,
<div><br>
</div>
<div>There are few files in which there were no errors while we reviewed using Jslint tool, but still it didn't passed the test and showed as 'failed' in dashboard. Could you please let us know what are the requirements that specific file should pass the test
on dashboard apart from removing the errors through Jslint tool.</div>
<div>Below are few files among them from 'geo' folder:</div>
<div>'command.js', 'feature.js', 'latlng.js'</div>
<div><br>
</div>
<div>Regards,</div>
<div>Keval<br>
<div style="font-size:16px;font-family:'Times New Roman'">
<hr>
<div style="direction:ltr"><font color="#000000" face="Tahoma"><b>From:</b> <a href="mailto:opengeoscience-developers-bounces@public.kitware.com" target="_blank">
opengeoscience-developers-bounces@public.kitware.com</a> [<a href="mailto:opengeoscience-developers-bounces@public.kitware.com" target="_blank">opengeoscience-developers-bounces@public.kitware.com</a>] on behalf of Aashish Chaudhary [<a href="mailto:aashish.chaudhary@kitware.com" target="_blank">aashish.chaudhary@kitware.com</a>]<br>
<b>Sent:</b> Thursday, April 25, 2013 3:47 AM<br>
<b>To:</b> opengeoscience-developers<br>
<b>Subject:</b> [Opengeoscience-developers] js-lint branch<br>
</font><br>
</div>
<div>
<div>
<div></div>
<div>
<div dir="ltr">Manas, Keval,
<div><br>
</div>
<div>What's the current status? Should I go ahead and review it? </div>
<div><br>
</div>
<div>- Aashish</div>
<div><br>
</div>
<div>
<div><br>
</div>
-- <br>
| Aashish Chaudhary <br>
| R&D Engineer <br>
| Kitware Inc. <br>
| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a> </div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
<br>
<br clear="all">
<div><br>
</div>
-- <br>
| Aashish Chaudhary <br>
| R&D Engineer <br>
| Kitware Inc. <br>
| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a> </div>
</div>
</div>
</div>
</div>
</div>
</div>
</div></div></div>
</div>
</div>
<br>_______________________________________________<br>
Opengeoscience-developers mailing list<br>
<a href="mailto:Opengeoscience-developers@public.kitware.com">Opengeoscience-developers@public.kitware.com</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers</a><br>
<br></blockquote></div><br></div></div>