user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

use @ instead of space for link ambiguity markers

Browse source 
since spaces aren't allowed in link targets in commonmark, a new symbol
is needed to separate the marker from the rest of the path. hence, @
This commit is contained in:
QuietMisdreavus 2018-01-08 09:23:12 -06:00 • committed by Manish Goregaokar
parent a3d71d7405
commit 4a20fb44c8
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/librustdoc/clean/mod.rs
View file

@ -848,21 +848,21 @@ impl Clean<Attributes> for [ast::Attribute] {
"trait", "union"].iter()
.find(|p| link.starts_with(**p)) {
kind = PathKind::Type;
link.trim_left_matches(prefix).trim()
link.trim_left_matches(prefix).trim_left_matches('@')
} else if let Some(prefix) =
["const", "static"].iter()
.find(|p| link.starts_with(**p)) {
kind = PathKind::Value;
link.trim_left_matches(prefix).trim()
link.trim_left_matches(prefix).trim_left_matches('@')
} else if link.ends_with("()") {
kind = PathKind::Value;
link.trim_right_matches("()").trim()
link.trim_right_matches("()")
} else if link.ends_with('!') {
kind = PathKind::Macro;
link.trim_right_matches('!').trim()
link.trim_right_matches('!')
} else {
link.trim()
};
&link[..]
}.trim();
// avoid resolving things (i.e. regular links) which aren't like paths
// FIXME(Manishearth) given that most links have slashes in them might be worth