Skip to content

Commit dbdd50b

Browse files
committed
With uglifyjs 1.0.6, if(false) branches removed, updating docs for next release.
1 parent 5c49b15 commit dbdd50b

2 files changed

Lines changed: 11 additions & 3 deletions

File tree

docs/optimization.html

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -276,9 +276,7 @@ <h2><a name="hasjs">Integration with has.js</a><span class="sectionMark">&sect;
276276
}
277277
</code></pre>
278278

279-
<p>Then, if you use the <b>optimize</b> setting that uses Closure Compiler (when run under Java), Closure Compiler will optimize out the dead code branch! So you can do custom builds of your code that are optimized for a set of has() tests.</p>
280-
281-
<span class="note">The default optimize setting, "uglify", does <strong>not</strong> do dead code branch removal at this time. It is hoped that UglifyJS will implement that feature at some point.</span>
279+
<p>Then, if you use the default optimize setting of "uglify" in r.js 0.26.0 or later, or if the <b>optimize</b> setting is set to "closure" (when <a href="https://github.com/jrburke/r.js/blob/master/README.md">run under Java</a>), the minifier will optimize out the dead code branch! So you can do custom builds of your code that are optimized for a set of has() tests.</p>
282280

283281
</div>
284282

tasks.txt

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,16 @@ Next release
88

99
- Document require for amd list
1010

11+
- Node 0.5.3 running ./alln.sh in r.js fails:
12+
Error: require.paths is removed. Use node_modules folders, or the NODE_PATH environment variable instead.
13+
at Function.<anonymous> (module.js:360:11)
14+
Figure out a better way, use a module that can be included: var r = require("r"); This
15+
would allow the directory to be the directory of the script...
16+
17+
18+
- Error messages not having useful lineNumber and fileName in Chrome:
19+
https://github.com/jrburke/requirejs/issues/116
20+
1121
- Update docs to mention npm install option for r.js.
1222

1323
Docs:

0 commit comments

Comments
 (0)