Skip to content

Commit 67b5129

Browse files
committed
Rollup merge of #21925 - sfackler:allow-missing-copy, r=alexcrichton
This was particularly helpful in the time just after OIBIT's implementation to make sure things that were supposed to be Copy continued to be, but it's now creates a lot of noise for types that intentionally don't want to be Copy. r? @alexcrichton
2 parents cfae247 + 85a85c2 commit 67b5129

File tree

28 files changed

+2
-61
lines changed

28 files changed

+2
-61
lines changed

src/doc/reference.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1813,7 +1813,6 @@ default visibility with the `priv` keyword. When an item is declared as `pub`,
18131813
it can be thought of as being accessible to the outside world. For example:
18141814

18151815
```
1816-
# #![allow(missing_copy_implementations)]
18171816
# fn main() {}
18181817
// Declare a private struct
18191818
struct Foo;

src/libcollections/string.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ pub struct FromUtf8Error {
4949

5050
/// A possible error value from the `String::from_utf16` function.
5151
#[stable(feature = "rust1", since = "1.0.0")]
52-
#[allow(missing_copy_implementations)]
5352
#[derive(Debug)]
5453
pub struct FromUtf16Error(());
5554

src/libcore/hash/sip.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ use super::{Hasher, Writer};
3030
/// strong, this implementation has not been reviewed for such purposes.
3131
/// As such, all cryptographic uses of this implementation are strongly
3232
/// discouraged.
33-
#[allow(missing_copy_implementations)]
3433
pub struct SipHasher {
3534
k0: u64,
3635
k1: u64,

src/libcore/marker.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -396,7 +396,6 @@ pub struct InvariantLifetime<'a>;
396396
reason = "likely to change with new variance strategy")]
397397
#[lang="no_copy_bound"]
398398
#[derive(Clone, PartialEq, Eq, PartialOrd, Ord)]
399-
#[allow(missing_copy_implementations)]
400399
pub struct NoCopy;
401400

402401
/// A type which is considered managed by the GC. This is typically
@@ -405,5 +404,4 @@ pub struct NoCopy;
405404
reason = "likely to change with new variance strategy")]
406405
#[lang="managed_bound"]
407406
#[derive(Clone, PartialEq, Eq, PartialOrd, Ord)]
408-
#[allow(missing_copy_implementations)]
409407
pub struct Managed;

src/libcore/str/mod.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,6 @@ impl FromStr for bool {
149149

150150
/// An error returned when parsing a `bool` from a string fails.
151151
#[derive(Debug, Clone, PartialEq)]
152-
#[allow(missing_copy_implementations)]
153152
#[stable(feature = "rust1", since = "1.0.0")]
154153
pub struct ParseBoolError { _priv: () }
155154

src/liblibc/lib.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -332,15 +332,12 @@ pub mod types {
332332
/// variants, because the compiler complains about the repr attribute
333333
/// otherwise.
334334
#[repr(u8)]
335-
#[allow(missing_copy_implementations)]
336335
pub enum c_void {
337336
__variant1,
338337
__variant2,
339338
}
340339

341-
#[allow(missing_copy_implementations)]
342340
pub enum FILE {}
343-
#[allow(missing_copy_implementations)]
344341
pub enum fpos_t {}
345342
}
346343
pub mod c99 {
@@ -354,9 +351,7 @@ pub mod types {
354351
pub type uint64_t = u64;
355352
}
356353
pub mod posix88 {
357-
#[allow(missing_copy_implementations)]
358354
pub enum DIR {}
359-
#[allow(missing_copy_implementations)]
360355
pub enum dirent_t {}
361356
}
362357
pub mod posix01 {}

src/librand/lib.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -387,7 +387,6 @@ pub trait SeedableRng<Seed>: Rng {
387387
/// [1]: Marsaglia, George (July 2003). ["Xorshift
388388
/// RNGs"](http://www.jstatsoft.org/v08/i14/paper). *Journal of
389389
/// Statistical Software*. Vol. 8 (Issue 14).
390-
#[allow(missing_copy_implementations)]
391390
#[derive(Clone)]
392391
pub struct XorShiftRng {
393392
x: u32,

src/librustc/lint/builtin.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2038,7 +2038,7 @@ declare_lint! {
20382038

20392039
declare_lint! {
20402040
pub MISSING_COPY_IMPLEMENTATIONS,
2041-
Warn,
2041+
Allow,
20422042
"detects potentially-forgotten implementations of `Copy`"
20432043
}
20442044

src/librustc/middle/infer/region_inference/mod.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,6 @@ pub struct RegionVarBindings<'a, 'tcx: 'a> {
224224
}
225225

226226
#[derive(Debug)]
227-
#[allow(missing_copy_implementations)]
228227
pub struct RegionSnapshot {
229228
length: uint,
230229
skolemization_count: u32,

src/librustc/middle/traits/coherence.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ pub fn impl_can_satisfy(infcx: &InferCtxt,
5959
|o| selcx.evaluate_obligation(o))
6060
}
6161

62-
#[allow(missing_copy_implementations)]
6362
pub enum OrphanCheckErr<'tcx> {
6463
NoLocalInputType,
6564
UncoveredTy(Ty<'tcx>),

0 commit comments

Comments
 (0)