gryf/pentadactyl-pm
1
0
Fork 0
mirror of https://github.com/gryf/pentadactyl-pm.git synced 2025-12-22 19:07:58 +01:00
Code Issues Projects Releases Wiki Activity

always specify the radix arg when calling parseInt rather than rely on it

Browse Source 
'guessing' correctly
This commit is contained in:
Doug Kearns
2007-11-07 11:16:53 +00:00
parent 14e4cf2f56
commit 18f6f3e8c6
5 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
content/buffers.js
View File

@@ -402,7 +402,7 @@ vimperator.Buffer = function() //{{{
var match; var match;
if (match = buffer.match(/^(\d+):?/)) if (match = buffer.match(/^(\d+):?/))
return vimperator.tabs.select(parseInt(match[1]) - 1, false); // make it zero-based return vimperator.tabs.select(parseInt(match[1], 10) - 1, false); // make it zero-based
var matches = []; var matches = [];
var lower_buffer = buffer.toLowerCase(); var lower_buffer = buffer.toLowerCase();

6
content/commands.js
View File

@@ -207,7 +207,7 @@ vimperator.Commands = function() //{{{
// parse count // parse count
if (matches[0]) if (matches[0])
matches[0] = parseInt(matches[0]); matches[0] = parseInt(matches[0], 10);
else else
matches[0] = -1; matches[0] = -1;
@@ -1301,7 +1301,7 @@ vimperator.Commands = function() //{{{
} }
else if (type == "number") else if (type == "number")
{ {
var num = parseInt(val, 10); var num = parseInt(val);
if (isNaN(num)) if (isNaN(num))
vimperator.echoerr("E521: Number required after =: " + option.name + "=" + val); vimperator.echoerr("E521: Number required after =: " + option.name + "=" + val);
else else
@@ -1483,7 +1483,7 @@ vimperator.Commands = function() //{{{
} }
else if (/^\d+$/.test(args)) else if (/^\d+$/.test(args))
{ {
var index = parseInt(args) - 1; var index = parseInt(args, 10) - 1;
if (index < vimperator.tabs.count()) if (index < vimperator.tabs.count())
vimperator.tabs.select(index, true); vimperator.tabs.select(index, true);
else else

4
content/tabs.js
View File

@@ -67,9 +67,9 @@ vimperator.Tabs = function() //{{{
else else
{ {
if (spec.match(/^([+-]\d+)$/)) // relative position +/-N if (spec.match(/^([+-]\d+)$/)) // relative position +/-N
position += parseInt(spec); position += parseInt(spec, 10);
else // absolute position else // absolute position
position = parseInt(spec); position = parseInt(spec, 10);
} }
if (position > last) if (position > last)

2
content/ui.js
View File

@@ -708,7 +708,7 @@ vimperator.CommandLine = function() //{{{
case "<LeftMouse>": case "<LeftMouse>":
if (event.originalTarget.className == "hl-URL buffer-list") if (event.originalTarget.className == "hl-URL buffer-list")
{ {
vimperator.tabs.select(parseInt(event.originalTarget.parentNode.parentNode.firstChild.textContent) - 1); vimperator.tabs.select(parseInt(event.originalTarget.parentNode.parentNode.firstChild.textContent, 10) - 1);
close_window = true; close_window = true;
break; break;
} }

2
content/vimperator.js
View File

@@ -311,7 +311,7 @@ const vimperator = (function() //{{{
// Number // Number
else if (match = string.match(/^(\d+)$/)) else if (match = string.match(/^(\d+)$/))
{ {
return parseInt(match[1]); return parseInt(match[1], 10);
} }
var reference = this.variableReference(string); var reference = this.variableReference(string);