Skip to content

Commit

Permalink
Auto merge of #51592 - GuillaumeGomez:fix-macro-doc-search, r=Guillau…
Browse files Browse the repository at this point in the history
…meGomez

Fix macro missing from doc search

Fixes #51095.

r? @QuietMisdreavus
  • Loading branch information
bors committed Jun 29, 2018
2 parents 57dd722 + e3b2583 commit 775ce97
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/librustdoc/html/static/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -465,10 +465,8 @@
var res = buildHrefAndPath(obj);
obj.displayPath = pathSplitter(res[0]);
obj.fullPath = obj.displayPath + obj.name;
if (obj.ty === TY_KEYWORD) {
// To be sure than it isn't considered as duplicate with items.
obj.fullPath += '|k';
}
// To be sure than it some items aren't considered as duplicate.
obj.fullPath += '|' + obj.ty;
obj.href = res[1];
out.push(obj);
if (out.length >= MAX_RESULTS) {
Expand Down
20 changes: 20 additions & 0 deletions src/test/rustdoc-js/macro-check.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
// Copyright 2018 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// ignore-order

const QUERY = 'panic';

const EXPECTED = {
'others': [
{ 'path': 'std', 'name': 'panic', ty: 14 }, // 15 is for macros
{ 'path': 'std', 'name': 'panic', ty: 0 }, // 0 is for modules
],
};

0 comments on commit 775ce97

Please sign in to comment.