Even better, just run 'ant jsHint' at the Cesium root level.  The ant task
is more reliable than the Eclipse plug-in.


On Mon, Apr 15, 2013 at 5:46 PM, Ed Mackey <notifications@github.com> wrote:

> In Apps/Sandcastle/CesiumSandcastle.js:
>
> >          var galleryTab = registry.byId('galleryContainer');
> > +        var innerPanel = registry.byId('innerPanel');
>
> Do you have Eclipse set up to show jsHints? There are some warnings here:
> unused variables on lines 672 & 673 (innerPanel is re-declared below), and
> another on line 899. Check out the info on setting up Eclipse for Cesium<https://github.com/AnalyticalGraphicsInc/cesium/wiki/Contributor%27s-Guide#set-up-eclipse>and then look for the little yellow exclamation marks that jsHint puts
> alongside the code.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/AnalyticalGraphicsInc/cesium/pull/635/files#r3802633>
> .
>