Skip to content

Commit

Permalink
Add notes to keep the UnusedExterns structs synced up
Browse files Browse the repository at this point in the history
  • Loading branch information
est31 committed Mar 8, 2021
1 parent d8c9a28 commit d018ef1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions compiler/rustc_errors/src/json.rs
Expand Up @@ -335,6 +335,10 @@ struct FutureIncompatReport {
future_incompat_report: Vec<FutureBreakageItem>,
}

// NOTE: Keep this in sync with the equivalent structs in rustdoc's
// doctest component (as well as cargo).
// We could unify this struct the one in rustdoc but they have different
// ownership semantics, so doing so would create wasteful allocations.
#[derive(Encodable)]
struct UnusedExterns<'a, 'b, 'c> {
/// The severity level of the unused dependencies lint
Expand Down
4 changes: 4 additions & 0 deletions src/librustdoc/doctest.rs
Expand Up @@ -278,6 +278,10 @@ impl DirState {
}
}

// NOTE: Keep this in sync with the equivalent structs in rustc
// and cargo.
// We could unify this struct the one in rustc but they have different
// ownership semantics, so doing so would create wasteful allocations.
#[derive(serde::Serialize, serde::Deserialize)]
struct UnusedExterns {
/// Lint level of the unused_crate_dependencies lint
Expand Down

0 comments on commit d018ef1

Please sign in to comment.