diff options
| author | Guillaume Gomez <guillaume.gomez@huawei.com> | 2021-11-29 10:20:11 +0100 |
|---|---|---|
| committer | Guillaume Gomez <guillaume.gomez@huawei.com> | 2021-11-30 22:53:20 +0100 |
| commit | b647c902620f862106da2baf6eb5575289cb447f (patch) | |
| tree | a800ff9f8c1eb9603011a2eec268a9536697c557 | |
| parent | 0c1b0f77576fe9b539a53322298dc26b75e255e8 (diff) | |
| download | rust-b647c902620f862106da2baf6eb5575289cb447f.tar.gz rust-b647c902620f862106da2baf6eb5575289cb447f.zip | |
Improve code by replacing &str with Symbol in render_stability_since_raw
| -rw-r--r-- | src/librustdoc/clean/types.rs | 8 | ||||
| -rw-r--r-- | src/librustdoc/html/render/mod.rs | 18 | ||||
| -rw-r--r-- | src/librustdoc/html/render/print_item.rs | 8 |
3 files changed, 15 insertions, 19 deletions
diff --git a/src/librustdoc/clean/types.rs b/src/librustdoc/clean/types.rs index 37acf68defd..aac7dbaca2b 100644 --- a/src/librustdoc/clean/types.rs +++ b/src/librustdoc/clean/types.rs @@ -602,16 +602,16 @@ impl Item { }) } - crate fn stable_since(&self, tcx: TyCtxt<'_>) -> Option<SymbolStr> { + crate fn stable_since(&self, tcx: TyCtxt<'_>) -> Option<Symbol> { match self.stability(tcx)?.level { - StabilityLevel::Stable { since, .. } => Some(since.as_str()), + StabilityLevel::Stable { since, .. } => Some(since), StabilityLevel::Unstable { .. } => None, } } - crate fn const_stable_since(&self, tcx: TyCtxt<'_>) -> Option<SymbolStr> { + crate fn const_stable_since(&self, tcx: TyCtxt<'_>) -> Option<Symbol> { match self.const_stability(tcx)?.level { - StabilityLevel::Stable { since, .. } => Some(since.as_str()), + StabilityLevel::Stable { since, .. } => Some(since), StabilityLevel::Unstable { .. } => None, } } diff --git a/src/librustdoc/html/render/mod.rs b/src/librustdoc/html/render/mod.rs index 5a2c1ea214d..eaeda4d178b 100644 --- a/src/librustdoc/html/render/mod.rs +++ b/src/librustdoc/html/render/mod.rs @@ -804,17 +804,17 @@ fn assoc_type( fn render_stability_since_raw( w: &mut Buffer, - ver: Option<&str>, + ver: Option<Symbol>, const_stability: Option<&ConstStability>, - containing_ver: Option<&str>, - containing_const_ver: Option<&str>, + containing_ver: Option<Symbol>, + containing_const_ver: Option<Symbol>, ) { let ver = ver.filter(|inner| !inner.is_empty()); match (ver, const_stability) { // stable and const stable (Some(v), Some(ConstStability { level: StabilityLevel::Stable { since }, .. })) - if Some(since.as_str()).as_deref() != containing_const_ver => + if Some(*since) != containing_const_ver => { write!( w, @@ -1651,23 +1651,19 @@ fn render_rightside( ) { let tcx = cx.tcx(); - let const_stable_since; // FIXME: Once https://github.com/rust-lang/rust/issues/67792 is implemented, we can remove // this condition. let (const_stability, const_stable_since) = match render_mode { - RenderMode::Normal => { - const_stable_since = containing_item.const_stable_since(tcx); - (item.const_stability(tcx), const_stable_since.as_deref()) - } + RenderMode::Normal => (item.const_stability(tcx), containing_item.const_stable_since(tcx)), RenderMode::ForDeref { .. } => (None, None), }; write!(w, "<div class=\"rightside\">"); render_stability_since_raw( w, - item.stable_since(tcx).as_deref(), + item.stable_since(tcx), const_stability, - containing_item.stable_since(tcx).as_deref(), + containing_item.stable_since(tcx), const_stable_since, ); diff --git a/src/librustdoc/html/render/print_item.rs b/src/librustdoc/html/render/print_item.rs index fc65b1d63cb..da66cd20d62 100644 --- a/src/librustdoc/html/render/print_item.rs +++ b/src/librustdoc/html/render/print_item.rs @@ -101,7 +101,7 @@ pub(super) fn print_item( let mut stability_since_raw = Buffer::new(); render_stability_since_raw( &mut stability_since_raw, - item.stable_since(cx.tcx()).as_deref(), + item.stable_since(cx.tcx()), item.const_stability(cx.tcx()), None, None, @@ -1462,10 +1462,10 @@ fn render_stability_since( ) { render_stability_since_raw( w, - item.stable_since(tcx).as_deref(), + item.stable_since(tcx), item.const_stability(tcx), - containing_item.stable_since(tcx).as_deref(), - containing_item.const_stable_since(tcx).as_deref(), + containing_item.stable_since(tcx), + containing_item.const_stable_since(tcx), ) } |
