Problem with duplicate class in Rhino and closure-compiler as well as several small fixes and cleanups

Hi Phil,
when I used your provided/rerleased jar everything worked well, but then I created my own one with Maven (without any changes) and I got IllegalAccessException/Error. The reason is that closure-compiler.jar contains the org.mozilla.classfile package with classes. While these classes are compatible with Rhino 1.7R3 they are not with Rhino 1.7R4 hence the exception. I can only assume that you use another OS to bundle the jar and as such the order of classes loaded when packaging the released jar is different from mine (Windows 7). I checked newer closure-compilers, but none is compatible with Rhino 1.7R4, so I had to change the POM to exclude the offending classes.

After this fix I went forward and removed some warnings and resolved two issues from your task tracker because I had the same problems. I am sorry that Git somehow messed up the diffs, all changes are minor I only changed a few characters in a line, but still it seems it changed the whole file due to line endings. I had thought we are past such issues... Sigh.

I created different commits for the different things so you can cherry-pick what you like. Thank you for ant-jshint.
Peter