about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2021-01-01 20:01:45 +0000
committerbors <bors@rust-lang.org>2021-01-01 20:01:45 +0000
commit17eec1433c69972844dd228b5fe801f218e118c3 (patch)
treefa44e67a9805cfaf52c7c2a171c3af60a01e8407
parent206ee1eea3467fd1d7f1efdbeafe27880897bb2c (diff)
parenta3958d039dc3dd7555b6b0896fd94b59e1475c94 (diff)
downloadrust-17eec1433c69972844dd228b5fe801f218e118c3.tar.gz
rust-17eec1433c69972844dd228b5fe801f218e118c3.zip
Auto merge of #80569 - notriddle:patch-3, r=jyn514
Use Array.prototype.filter instead of open-coding

Part of #79052, originally suggested in https://github.com/rust-lang/rust/pull/79052#discussion_r523468743 by `@jyn514`

Besides making main.js smaller (always a plus), this also performs better by using the optimized filter implementation in your browser's JavaScript engine (according to `@GuillaumeGomez,` an 84% performance improvement).
-rw-r--r--src/librustdoc/html/static/main.js8
1 files changed, 1 insertions, 7 deletions
diff --git a/src/librustdoc/html/static/main.js b/src/librustdoc/html/static/main.js
index 1de4b0016c5..4d44b85ad65 100644
--- a/src/librustdoc/html/static/main.js
+++ b/src/librustdoc/html/static/main.js
@@ -667,13 +667,7 @@ function defocusSearchBar() {
                 results = {}, results_in_args = {}, results_returned = {},
                 split = valLower.split("::");
 
-            var length = split.length;
-            for (var z = 0; z < length; ++z) {
-                if (split[z] === "") {
-                    split.splice(z, 1);
-                    z -= 1;
-                }
-            }
+            split = split.filter(function(segment) { return segment !== ""; });
 
             function transformResults(results, isType) {
                 var out = [];