https://wiki.greasespot.net/api.php?action=feedcontributions&user=81.225.78.42&feedformat=atomGreaseSpot Wiki - User contributions [en]2024-03-29T13:49:46ZUser contributionsMediaWiki 1.41.0https://wiki.greasespot.net/index.php?title=Talk:Security_tips&diff=1768Talk:Security tips2007-03-20T11:27:45Z<p>81.225.78.42: </p>
<hr />
<div>I propose un-doing [http://wiki.greasespot.net/index.php?title=Security_tips&curid=1532&diff=1752&oldid=1751&rcid=254 this revision]. GreaseMonkey scripts always have a wrapper function. See [http://greasemonkey.devjavu.com/projects/greasemonkey/browser/trunk/src/components/greasemonkey.js greasemonkey.js] line 250.<br />
[[User:Arantius|Arantius]] 15:26, 15 March 2007 (EDT)<br />
<br />
: -1. The fact that Greasemonkey wraps scripts in an anonymous function is an irrelevant arcanum. <code>return</code> outside a function, even though it eventually ends up inside a function, is unclear. --[[User:Ldrhcp|Ldrhcp]] 18:16, 15 March 2007 (EDT)<br />
<br />
: +-0. I made that change (the one that was undone in the linked revision). While I think that bailing early is good and needless nesting is bad, I also think perhaps the nesting way is more clear, as an example. It also fits better with the fact that you might well have additional "else if" branches. We ''could'' give two examples, but that'd probably hurt more than it'd help. --[[User:81.225.78.42|81.225.78.42]] 07:27, 20 March 2007 (EDT)</div>81.225.78.42