diff --git a/chrome/content/vimperator/commands.js b/chrome/content/vimperator/commands.js index 70d58e2d..62bddeac 100644 --- a/chrome/content/vimperator/commands.js +++ b/chrome/content/vimperator/commands.js @@ -623,7 +623,7 @@ function Commands() //{{{ addDefaultCommand(new Command(["mapc[lear]"], function(args) { - if (args.length > 0) + if (args) { vimperator.echoerr("E474: Invalid argument"); return; @@ -667,7 +667,7 @@ function Commands() //{{{ function(args) { // ignore invalid mark characters unless there are no valid mark chars - if (args.length > 0 && !/[a-zA-Z]/.test(args)) + if (args && !/[a-zA-Z]/.test(args)) { vimperator.echoerr("E283: No marks matching \"" + args + "\""); return; @@ -756,8 +756,10 @@ function Commands() //{{{ addDefaultCommand(new Command(["o[pen]", "e[dit]"], function(args, special) { - if (args.length > 0) + if (args) + { vimperator.open(args); + } else { if (special) @@ -780,10 +782,10 @@ function Commands() //{{{ "and the first word is the name of a search engine (:open wikipedia linus torvalds " + "will open the wikipedia entry for linus torvalds). The short name of a search engine is automatically guessed from its name. " + "If you want to set a custom name, open the $FIREFOX_PROFILE/searchplugins/*.xml file of the search engine, and add/change " + - "<Alias>myalias</Alias> " + - "
  • Opened with the default search engine or keyword (specified with the 'defsearch' option) " + + "<Alias>myalias</Alias>
  • " + + "
  • Opened with the default search engine or keyword (specified with the 'defsearch' option) " + "if the first word is no search engine (:open linus torvalds will open a Google search for linux torvalds).
  • " + - "
  • Passed directly to Firefox in all other cases (:open www.osnews.com, www.slashdot.org will " + + "
  • Passed directly to Firefox in all other cases (:open www.osnews.com, www.slashdot.org will " + "open OSNews in the current, and Slashdot in a new background tab).
  • " + "" + "You WILL be able to use :open [-T \"linux\"] torvalds<Tab> to complete bookmarks " + @@ -838,7 +840,7 @@ function Commands() //{{{ function(args) { // ignore invalid mark characters unless there are no valid mark chars - if (args.length > 0 && !/[a-zA-Z0-9]/.test(args)) + if (args && !/[a-zA-Z0-9]/.test(args)) { vimperator.echoerr("E283: No QuickMarks matching \"" + args + "\""); return; @@ -1143,7 +1145,7 @@ function Commands() //{{{ if (/\btabopen\b/.test(vimperator.options["activate"])) where = special ? vimperator.NEW_BACKGROUND_TAB : vimperator.NEW_TAB; - if (args.length > 0) + if (args) vimperator.open(args, where); else vimperator.open("about:blank", where);