Rollup merge of #104459 - notriddle:notriddle/main-js-iife, r=GuillaumeGomez

matthiaskrgr · web-flow · commit baf1cbb70d21 · 2022-11-16T15:39:47.000+01:00
rustdoc: remove unused JS IIFE from main.js This [IIFE] made sense when it was added in f0683f9 and there was a local variable scoped to it, but now it only sets two globals, so it does nothing. [IIFE]: https://developer.mozilla.org/en-US/docs/Glossary/IIFE "immediately invoked function expression"
diff --git a/src/librustdoc/html/static/js/main.js b/src/librustdoc/html/static/js/main.js
@@ -47,10 +47,8 @@ function blurHandler(event, parentElem, hideCallback) {
     }
 }
 
-(function() {
-    window.rootPath = getVar("root-path");
-    window.currentCrate = getVar("current-crate");
-}());
+window.rootPath = getVar("root-path");
+window.currentCrate = getVar("current-crate");
 
 function setMobileTopbar() {
     // FIXME: It would be nicer to generate this text content directly in HTML,

Original file line number	Diff line number	Diff line change
`@@ -47,10 +47,8 @@ function blurHandler(event, parentElem, hideCallback) {`
`47`	`47`	`}`
`48`	`48`	`}`
`49`	`49`
`50`		`-(function() {`
`51`		`- window.rootPath = getVar("root-path");`
`52`		`- window.currentCrate = getVar("current-crate");`
`53`		`-}());`
	`50`	`+window.rootPath = getVar("root-path");`
	`51`	`+window.currentCrate = getVar("current-crate");`
`54`	`52`
`55`	`53`	`function setMobileTopbar() {`
`56`	`54`	`// FIXME: It would be nicer to generate this text content directly in HTML,`