Merge pull request #216 from superfell/vec_order

Fix BitVec to BitSlice partial_cmp. Fixes #215

src/vec/tests.rs

@@ -79,3 +79,19 @@ fn misc() {
 
 	assert_eq!(!bitvec![0, 1], bits![1, 0]);
 }
+
+#[test]
+fn bitvec_order() {
+	let k = bitvec![0, 1, 0, 1];
+	let r = bitvec![1, 0, 0, 0];
+	let k_slice = &k[..];
+	let r_slice = &r[..];
+	assert!(r > k);
+	assert!(k < r);
+	assert!(r_slice > k_slice);
+	assert!(k_slice < r_slice);
+	assert!(r_slice > k);
+	assert!(k < r_slice);
+	assert!(k_slice < r);
+	assert!(r > k_slice);
+}

src/vec/traits.rs

@@ -198,7 +198,7 @@ where
 {
 	#[inline]
 	fn partial_cmp(&self, other: &Rhs) -> Option<cmp::Ordering> {
-		other.partial_cmp(self.as_bitslice())
+		other.partial_cmp(self.as_bitslice()).map(|o| o.reverse())
 	}
 }