Skip to content

Commit

Permalink
Remove Copy and Clone impls for LocalInternedString.
Browse files Browse the repository at this point in the history
They aren't used.
  • Loading branch information
nnethercote committed Oct 18, 2019
1 parent 3532863 commit 0879f63
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/libsyntax_pos/symbol.rs
Expand Up @@ -1106,8 +1106,8 @@ fn with_interner<T, F: FnOnce(&mut Interner) -> T>(f: F) -> T {
}

/// An alternative to `Symbol` and `InternedString`, useful when the chars
/// within the symbol need to be accessed. It is best used for temporary
/// values.
/// within the symbol need to be accessed. It deliberately has limited
/// functionality and should only be used for temporary values.
///
/// Because the interner outlives any thread which uses this type, we can
/// safely treat `string` which points to interner data, as an immortal string,
Expand All @@ -1116,7 +1116,7 @@ fn with_interner<T, F: FnOnce(&mut Interner) -> T>(f: F) -> T {
// FIXME: ensure that the interner outlives any thread which uses
// `LocalInternedString`, by creating a new thread right after constructing the
// interner.
#[derive(Clone, Copy, Eq, PartialOrd, Ord)]
#[derive(Eq, PartialOrd, Ord)]
pub struct LocalInternedString {
string: &'static str,
}
Expand Down

0 comments on commit 0879f63

Please sign in to comment.