jsrrdgraph: Fixed jshint warnings/errors

Many fall in the category missing semicolon, but there are legitimate
bugs (like throwing an error with an undefined variable, using
isInfinite instead of !isFinite or fabs instead of Math.abs).

At some places, I moved the variable declarations to avoid duplicate
definition warnings.

Redundant breaks have been removed (after return / throw).

Global variables were implicitly defined in RrdDataFile (which caught my
attention) and Base64, these have been made local.

Also fixed some whitespace errors. Yay, the consistency. Not all (style)
issues are fixed.

1 files changed, 2 insertions, 2 deletions

diff --git a/js/RrdRpn.js b/js/RrdRpn.js
index cbb5e8f..5ca8daf 100644
--- a/js/RrdRpn.js
+++ b/js/RrdRpn.js
@@ -478,7 +478,7 @@ RrdRpn.prototype.calc = function (data_idx, output, output_idx)
                                 break;
                         case RrdRpn.OP_ISINF:
                                 if(stptr < 0) throw new RrdRpnError();
-                                this.rpnstack[stptr] = isInfinite(this.rpnstack[stptr]) ? 1.0 : 0.0;
+                                this.rpnstack[stptr] = !isFinite(this.rpnstack[stptr]) ? 1.0 : 0.0;
                                 break;
                         case RrdRpn.OP_SQRT:
                                 if(stptr < 0) throw new RrdRpnError();
@@ -603,7 +603,7 @@ RrdRpn.prototype.calc = function (data_idx, output, output_idx)
                                 break;
                         case RrdRpn.OP_ABS:
                                 if(stptr < 0) throw new RrdRpnError();
-                                this.rpnstack[stptr] = fabs(this.rpnstack[stptr]);
+                                this.rpnstack[stptr] = Math.abs(this.rpnstack[stptr]);
                                 break;
                         case RrdRpn.OP_END:
                                 break;