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

Simplify AttributesExt::cfg function and remove error emissions since they are not useful

Browse source
This commit is contained in:
Guillaume Gomez 2021-10-06 20:06:33 +02:00
parent 8fac41a530
commit 09c76884bc
3 changed files with 45 additions and 27 deletions
Download patch file Download diff file Expand all files Collapse all files

37
src/librustdoc/clean/types.rs
View file

@ -776,48 +776,31 @@ impl AttributesExt for [ast::Attribute] {
let sess = tcx.sess;
let doc_cfg_active = tcx.features().doc_cfg;
trait SingleExt {
type Item;
fn single(self) -> Option<Self::Item>;
}
impl<T: IntoIterator> SingleExt for T {
type Item = T::Item;
fn single(self) -> Option<Self::Item> {
let mut iter = self.into_iter();
let item = iter.next()?;
if iter.next().is_some() {
return None;
}
Some(item)
fn single<T: IntoIterator>(it: T) -> Option<T::Item> {
let mut iter = it.into_iter();
let item = iter.next()?;
if iter.next().is_some() {
return None;
}
Some(item)
}
let mut cfg = if doc_cfg_active {
let mut doc_cfg = self
.iter()
.filter(|attr| attr.has_name(sym::doc))
.filter_map(|attr| Some(attr.meta_item_list()?.single()?))
.flat_map(|attr| attr.meta_item_list().unwrap_or_else(Vec::new))
.filter(|attr| attr.has_name(sym::cfg))
.filter_map(|attr| Some(attr.meta_item_list()?.single()?.meta_item()?.clone()))
.peekable();
if doc_cfg.peek().is_some() {
doc_cfg
.filter_map(|attr| {
Cfg::parse(&attr)
.map_err(|e| sess.diagnostic().span_err(e.span, e.msg))
.ok()
})
.filter_map(|attr| Cfg::parse(attr.meta_item()?).ok())
.fold(Cfg::True, |cfg, new_cfg| cfg & new_cfg)
} else {
self.iter()
.filter(|attr| attr.has_name(sym::cfg))
.filter_map(|attr| Some(attr.meta_item_list()?.single()?.meta_item()?.clone()))
.filter_map(|attr| {
Cfg::parse(&attr)
.map_err(|e| sess.diagnostic().span_err(e.span, e.msg))
.ok()
})
.filter_map(|attr| single(attr.meta_item_list()?))
.filter_map(|attr| Cfg::parse(attr.meta_item()?).ok())
.filter(|cfg| !hidden_cfg.contains(cfg))
.fold(Cfg::True, |cfg, new_cfg| cfg & new_cfg)
}