fix some typos in comments
Signed-off-by: cui fliter <imcusg@gmail.com>
This commit is contained in:
parent
229e875878
commit
442f848d74
13 changed files with 15 additions and 15 deletions
|
@ -376,7 +376,7 @@ impl<'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> {
|
||||||
|
|
||||||
/// Read an immediate from a place, asserting that that is possible with the given layout.
|
/// Read an immediate from a place, asserting that that is possible with the given layout.
|
||||||
///
|
///
|
||||||
/// If this suceeds, the `ImmTy` is never `Uninit`.
|
/// If this succeeds, the `ImmTy` is never `Uninit`.
|
||||||
#[inline(always)]
|
#[inline(always)]
|
||||||
pub fn read_immediate(
|
pub fn read_immediate(
|
||||||
&self,
|
&self,
|
||||||
|
|
|
@ -333,7 +333,7 @@ fn expand_macro<'cx>(
|
||||||
assert!(try_success_result.is_err(), "Macro matching returned a success on the second try");
|
assert!(try_success_result.is_err(), "Macro matching returned a success on the second try");
|
||||||
|
|
||||||
if let Some(result) = tracker.result {
|
if let Some(result) = tracker.result {
|
||||||
// An irrecoverable error occured and has been emitted.
|
// An irrecoverable error occurred and has been emitted.
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -147,7 +147,7 @@ pub enum AttributeDuplicates {
|
||||||
FutureWarnPreceding,
|
FutureWarnPreceding,
|
||||||
}
|
}
|
||||||
|
|
||||||
/// A conveniece macro to deal with `$($expr)?`.
|
/// A convenience macro to deal with `$($expr)?`.
|
||||||
macro_rules! or_default {
|
macro_rules! or_default {
|
||||||
($default:expr,) => {
|
($default:expr,) => {
|
||||||
$default
|
$default
|
||||||
|
|
|
@ -35,7 +35,7 @@ struct ClosureSignatures<'tcx> {
|
||||||
bound_sig: ty::PolyFnSig<'tcx>,
|
bound_sig: ty::PolyFnSig<'tcx>,
|
||||||
/// The signature within the function body.
|
/// The signature within the function body.
|
||||||
/// This mostly differs in the sense that lifetimes are now early bound and any
|
/// This mostly differs in the sense that lifetimes are now early bound and any
|
||||||
/// opaque types from the signature expectation are overriden in case there are
|
/// opaque types from the signature expectation are overridden in case there are
|
||||||
/// explicit hidden types written by the user in the closure signature.
|
/// explicit hidden types written by the user in the closure signature.
|
||||||
liberated_sig: ty::FnSig<'tcx>,
|
liberated_sig: ty::FnSig<'tcx>,
|
||||||
}
|
}
|
||||||
|
|
|
@ -360,7 +360,7 @@ fn lint_int_literal<'tcx>(
|
||||||
}
|
}
|
||||||
|
|
||||||
if lint_overflowing_range_endpoint(cx, lit, v, max, e, t.name_str()) {
|
if lint_overflowing_range_endpoint(cx, lit, v, max, e, t.name_str()) {
|
||||||
// The overflowing literal lint was emited by `lint_overflowing_range_endpoint`.
|
// The overflowing literal lint was emitted by `lint_overflowing_range_endpoint`.
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -429,7 +429,7 @@ fn lint_uint_literal<'tcx>(
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if lint_overflowing_range_endpoint(cx, lit, lit_val, max, e, t.name_str()) {
|
if lint_overflowing_range_endpoint(cx, lit, lit_val, max, e, t.name_str()) {
|
||||||
// The overflowing literal lint was emited by `lint_overflowing_range_endpoint`.
|
// The overflowing literal lint was emitted by `lint_overflowing_range_endpoint`.
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
if let Some(repr_str) = get_bin_hex_repr(cx, lit) {
|
if let Some(repr_str) = get_bin_hex_repr(cx, lit) {
|
||||||
|
|
|
@ -1541,7 +1541,7 @@ impl<'tcx> Place<'tcx> {
|
||||||
/// If MirPhase >= Derefered and if projection contains Deref,
|
/// If MirPhase >= Derefered and if projection contains Deref,
|
||||||
/// It's guaranteed to be in the first place
|
/// It's guaranteed to be in the first place
|
||||||
pub fn has_deref(&self) -> bool {
|
pub fn has_deref(&self) -> bool {
|
||||||
// To make sure this is not accidently used in wrong mir phase
|
// To make sure this is not accidentally used in wrong mir phase
|
||||||
debug_assert!(
|
debug_assert!(
|
||||||
self.projection.is_empty() || !self.projection[1..].contains(&PlaceElem::Deref)
|
self.projection.is_empty() || !self.projection[1..].contains(&PlaceElem::Deref)
|
||||||
);
|
);
|
||||||
|
|
|
@ -85,7 +85,7 @@ pub enum MirPhase {
|
||||||
///
|
///
|
||||||
/// Also note that the lint pass which reports eg `200_u8 + 200_u8` as an error is run as a part
|
/// Also note that the lint pass which reports eg `200_u8 + 200_u8` as an error is run as a part
|
||||||
/// of analysis to runtime MIR lowering. To ensure lints are reported reliably, this means that
|
/// of analysis to runtime MIR lowering. To ensure lints are reported reliably, this means that
|
||||||
/// transformations which may supress such errors should not run on analysis MIR.
|
/// transformations which may suppress such errors should not run on analysis MIR.
|
||||||
Runtime(RuntimePhase),
|
Runtime(RuntimePhase),
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -118,7 +118,7 @@ impl<'a, 'tcx> Builder<'a, 'tcx> {
|
||||||
else_block: Some(else_block),
|
else_block: Some(else_block),
|
||||||
} => {
|
} => {
|
||||||
// When lowering the statement `let <pat> = <expr> else { <else> };`,
|
// When lowering the statement `let <pat> = <expr> else { <else> };`,
|
||||||
// the `<else>` block is nested in the parent scope enclosing this statment.
|
// the `<else>` block is nested in the parent scope enclosing this statement.
|
||||||
// That scope is usually either the enclosing block scope,
|
// That scope is usually either the enclosing block scope,
|
||||||
// or the remainder scope of the last statement.
|
// or the remainder scope of the last statement.
|
||||||
// This is to make sure that temporaries instantiated in `<expr>` are dropped
|
// This is to make sure that temporaries instantiated in `<expr>` are dropped
|
||||||
|
|
|
@ -72,7 +72,7 @@ impl<'r, 'a> EffectiveVisibilitiesVisitor<'r, 'a> {
|
||||||
update(node_id);
|
update(node_id);
|
||||||
if let ImportKind::Single { additional_ids: (id1, id2), .. } = import.kind {
|
if let ImportKind::Single { additional_ids: (id1, id2), .. } = import.kind {
|
||||||
// In theory all the single import IDs have individual visibilities and
|
// In theory all the single import IDs have individual visibilities and
|
||||||
// effective visibilities, but in practice these IDs go straigth to HIR
|
// effective visibilities, but in practice these IDs go straight to HIR
|
||||||
// where all their few uses assume that their (effective) visibility
|
// where all their few uses assume that their (effective) visibility
|
||||||
// applies to the whole syntactic `use` item. So they all get the same
|
// applies to the whole syntactic `use` item. So they all get the same
|
||||||
// value which is the maximum of all bindings. Maybe HIR for imports
|
// value which is the maximum of all bindings. Maybe HIR for imports
|
||||||
|
|
|
@ -114,7 +114,7 @@ pub enum Lld {
|
||||||
/// relevant now.
|
/// relevant now.
|
||||||
///
|
///
|
||||||
/// The second goal is to keep the number of flavors to the minimum if possible.
|
/// The second goal is to keep the number of flavors to the minimum if possible.
|
||||||
/// LLD somewhat forces our hand here because that linker is self-sufficent only if its executable
|
/// LLD somewhat forces our hand here because that linker is self-sufficient only if its executable
|
||||||
/// (`argv[0]`) is named in specific way, otherwise it doesn't work and requires a
|
/// (`argv[0]`) is named in specific way, otherwise it doesn't work and requires a
|
||||||
/// `-flavor LLD_FLAVOR` argument to choose which logic to use. Our shipped `rust-lld` in
|
/// `-flavor LLD_FLAVOR` argument to choose which logic to use. Our shipped `rust-lld` in
|
||||||
/// particular is not named in such specific way, so it needs the flavor option, so we make our
|
/// particular is not named in such specific way, so it needs the flavor option, so we make our
|
||||||
|
|
|
@ -674,7 +674,7 @@ impl FromWithTcx<clean::Variant> for Variant {
|
||||||
impl FromWithTcx<clean::Discriminant> for Discriminant {
|
impl FromWithTcx<clean::Discriminant> for Discriminant {
|
||||||
fn from_tcx(disr: clean::Discriminant, tcx: TyCtxt<'_>) -> Self {
|
fn from_tcx(disr: clean::Discriminant, tcx: TyCtxt<'_>) -> Self {
|
||||||
Discriminant {
|
Discriminant {
|
||||||
// expr is only none if going throught the inlineing path, which gets
|
// expr is only none if going through the inlineing path, which gets
|
||||||
// `rustc_middle` types, not `rustc_hir`, but because JSON never inlines
|
// `rustc_middle` types, not `rustc_hir`, but because JSON never inlines
|
||||||
// the expr is always some.
|
// the expr is always some.
|
||||||
expr: disr.expr(tcx).unwrap(),
|
expr: disr.expr(tcx).unwrap(),
|
||||||
|
|
|
@ -277,7 +277,7 @@ impl<'tcx> FormatRenderer<'tcx> for JsonRenderer<'tcx> {
|
||||||
|
|
||||||
let e = ExternalCrate { crate_num: LOCAL_CRATE };
|
let e = ExternalCrate { crate_num: LOCAL_CRATE };
|
||||||
|
|
||||||
// FIXME(adotinthevoid): Remove this, as it's not consistant with not
|
// FIXME(adotinthevoid): Remove this, as it's not consistent with not
|
||||||
// inlining foreign items.
|
// inlining foreign items.
|
||||||
let foreign_trait_items = self.get_trait_items();
|
let foreign_trait_items = self.get_trait_items();
|
||||||
let mut index = (*self.index).clone().into_inner();
|
let mut index = (*self.index).clone().into_inner();
|
||||||
|
|
|
@ -53,7 +53,7 @@ pub struct ItemSummary {
|
||||||
/// `["std", "io", "lazy", "Lazy"]` for `std::io::lazy::Lazy`).
|
/// `["std", "io", "lazy", "Lazy"]` for `std::io::lazy::Lazy`).
|
||||||
///
|
///
|
||||||
/// Note that items can appear in multiple paths, and the one chosen is implementation
|
/// Note that items can appear in multiple paths, and the one chosen is implementation
|
||||||
/// defined. Currenty, this is the full path to where the item was defined. Eg
|
/// defined. Currently, this is the full path to where the item was defined. Eg
|
||||||
/// [`String`] is currently `["alloc", "string", "String"]` and [`HashMap`] is
|
/// [`String`] is currently `["alloc", "string", "String"]` and [`HashMap`] is
|
||||||
/// `["std", "collections", "hash", "map", "HashMap"]`, but this is subject to change.
|
/// `["std", "collections", "hash", "map", "HashMap"]`, but this is subject to change.
|
||||||
pub path: Vec<String>,
|
pub path: Vec<String>,
|
||||||
|
@ -351,7 +351,7 @@ pub enum Variant {
|
||||||
/// A variant with unnamed fields.
|
/// A variant with unnamed fields.
|
||||||
///
|
///
|
||||||
/// Unlike most of json, `#[doc(hidden)]` fields will be given as `None`
|
/// Unlike most of json, `#[doc(hidden)]` fields will be given as `None`
|
||||||
/// instead of being ommited, because order matters.
|
/// instead of being omitted, because order matters.
|
||||||
///
|
///
|
||||||
/// ```rust
|
/// ```rust
|
||||||
/// enum Demo {
|
/// enum Demo {
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue