Fix for opt in Copy

Author: sfackler

src/lib.rs

@@ -202,6 +202,7 @@ pub trait NoticeHandler: Send {
 /// A notice handler which logs at the `info` level.
 ///
 /// This is the default handler used by a `Connection`.
+#[deriving(Copy)]
 pub struct DefaultNoticeHandler;
 
 impl NoticeHandler for DefaultNoticeHandler {
@@ -319,6 +320,7 @@ impl<'conn> Notifications<'conn> {
 }
 
 /// Contains information necessary to cancel queries for a session
+#[deriving(Copy)]
 pub struct CancelData {
     /// The process ID of the session
     pub process_id: u32,

src/types/array.rs

@@ -7,7 +7,7 @@ use self::ArrayParent::{SliceParent, MutSliceParent, BaseParent};
 use self::MutArrayParent::{MutSliceMutParent, MutBaseParent};
 
 /// Information about a dimension of an array
-#[deriving(PartialEq, Eq, Clone)]
+#[deriving(PartialEq, Eq, Clone, Copy)]
 pub struct DimensionInfo {
     /// The size of the dimension
     pub len: uint,

src/types/range.rs

@@ -133,7 +133,7 @@ bounded_normalizable!(i32)
 bounded_normalizable!(i64)
 
 /// The possible sides of a bound
-#[deriving(PartialEq, Eq)]
+#[deriving(PartialEq, Eq, Copy)]
 pub enum BoundSide {
     /// An upper bound
     Upper,
@@ -150,9 +150,11 @@ pub trait BoundSided {
 }
 
 /// A tag type representing an upper bound
+#[allow(missing_copy_implementations)]
 pub enum UpperBound {}
 
 /// A tag type representing a lower bound
+#[allow(missing_copy_implementations)]
 pub enum LowerBound {}
 
 impl BoundSided for UpperBound {
@@ -168,7 +170,7 @@ impl BoundSided for LowerBound {
 }
 
 /// The type of a range bound
-#[deriving(PartialEq, Eq, Clone)]
+#[deriving(PartialEq, Eq, Clone, Copy)]
 pub enum BoundType {
     /// The bound includes its value
     Inclusive,
@@ -186,6 +188,8 @@ pub struct RangeBound<S: BoundSided, T> {
     pub type_: BoundType
 }
 
+impl<S, T> Copy for RangeBound<S, T> where S: BoundSided, T: Copy {}
+
 impl<S, T> Clone for RangeBound<S, T> where S: BoundSided, T: Clone {
     fn clone(&self) -> RangeBound<S, T> {
         RangeBound {
@@ -280,24 +284,24 @@ impl<'a, S, T> PartialEq for OptBound<'a, S, T> where S: BoundSided, T: PartialE
 
 impl<'a, S, T> PartialOrd for OptBound<'a, S, T> where S: BoundSided, T: PartialOrd {
     fn partial_cmp(&self, other: &OptBound<'a, S, T>) -> Option<Ordering> {
-        match (*self, *other, BoundSided::side(None::<S>)) {
-            (OptBound(None), OptBound(None), _) => Some(Equal),
-            (OptBound(None), _, Lower)
-            | (_, OptBound(None), Upper) => Some(Less),
-            (OptBound(None), _, Upper)
-            | (_, OptBound(None), Lower) => Some(Greater),
-            (OptBound(Some(a)), OptBound(Some(b)), _) => a.partial_cmp(b)
+        match (self, other, BoundSided::side(None::<S>)) {
+            (&OptBound(None), &OptBound(None), _) => Some(Equal),
+            (&OptBound(None), _, Lower)
+            | (_, &OptBound(None), Upper) => Some(Less),
+            (&OptBound(None), _, Upper)
+            | (_, &OptBound(None), Lower) => Some(Greater),
+            (&OptBound(Some(a)), &OptBound(Some(b)), _) => a.partial_cmp(b)
         }
     }
 }
 
 /// Represents a range of values.
-#[deriving(PartialEq, Eq, Clone)]
+#[deriving(PartialEq, Eq, Clone, Copy)]
 pub struct Range<T> {
     inner: InnerRange<T>,
 }
 
-#[deriving(PartialEq, Eq, Clone)]
+#[deriving(PartialEq, Eq, Clone, Copy)]
 enum InnerRange<T> {
     Empty,
     Normal(Option<RangeBound<LowerBound, T>>,

tests/types/uuid.rs

@@ -17,6 +17,6 @@ fn test_uuidarray_params() {
     let (v1, s1) = make_check("a0eebc99-9c0b-4ef8-bb6d-6bb9bd380a11");
     let (v2, s2) = make_check("00000000-0000-0000-0000-000000000000");
     let (v3, s3) = make_check("a0eebc99-9c0b-4ef8-bb6d-6bb9bd380a11");
-    test_array_params!("UUID", v1, s1, v2, s2, v3, s3);
+    test_array_params!("UUID", v1.clone(), s1, v2.clone(), s2, v3.clone(), s3);
 }