Changeset 675

Show
Ignore:
Timestamp:
03/10/08 12:26:33 (9 months ago)
Author:
oyasu..@gmail.com
Message:

Form fixes suggested by Anthony: braces for indentation of all loop constructs, and deploying GM_scriptMatchesUrl everywhere it was intended for.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/src/chrome/chromeFiles/content/browser.js

    r674 r675  
    450450  function scriptsMatching(urls) { 
    451451    var matching = []; 
    452     for (var i = 0, script = null; script = config.scripts[i]; i++) 
    453       for (var j = 0; j < urls.length; j++) 
     452    for (var i = 0, script = null; script = config.scripts[i]; i++) { 
     453      for (var j = 0; j < urls.length; j++) { 
    454454        if (GM_scriptMatchesUrl(script, urls[j])) { 
    455455          matching.push(script); 
    456456          break; 
    457457        } 
     458      } 
     459    } 
    458460    return matching; 
    459461  } 
    …… 
    490492  for (var i = 0; i < runsOnTop.length; i++) { 
    491493    var j = 0, item = runsOnTop[i]; 
    492     while (j < runsFramed.length) 
    493       if (item === runsFramed[j]) 
     494    while (j < runsFramed.length) { 
     495      if (item === runsFramed[j]) { 
    494496        runsFramed.splice(j, 1); 
    495       else 
     497      } else { 
    496498        j++; 
     499      } 
     500    } 
    497501  } 
    498502 

  • trunk/src/components/greasemonkey.js

    r651 r675  
    234234    config.load(); 
    235235 
    236     outer: 
    237     for (var i = 0; i < config.scripts.length; i++) { 
    238       var script = config.scripts[i]; 
    239       if (script.enabled) { 
    240         for (var j = 0; j < script.includes.length; j++) { 
    241           var pattern = convert2RegExp(script.includes[j]); 
    242  
    243           if (pattern.test(url)) { 
    244             for (var k = 0; k < script.excludes.length; k++) { 
    245               pattern = convert2RegExp(script.excludes[k]); 
    246  
    247               if (pattern.test(url)) { 
    248                 continue outer; 
    249               } 
    250             } 
    251  
    252             scripts.push(script); 
    253  
    254             continue outer; 
    255           } 
    256         } 
     236    for (var i = 0; script = config.scripts[i]; i++) { 
     237      if (script.enabled && GM_scriptMatchesUrl(script, url)) { 
     238        scripts.push(script); 
    257239      } 
    258240    }