Skip to content

Commit 0990272

Browse files
committed
---
yaml --- r: 152371 b: refs/heads/try2 c: 7580ef9 h: refs/heads/master i: 152369: 81bcf64 152367: f449d75 v: v3
1 parent 5ad9b4c commit 0990272

File tree

9 files changed

+209
-95
lines changed

9 files changed

+209
-95
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ refs/heads/snap-stage3: 78a7676898d9f80ab540c6df5d4c9ce35bb50463
55
refs/heads/try: 519addf6277dbafccbb4159db4b710c37eaa2ec5
66
refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
77
refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
8-
refs/heads/try2: 90339ef18919be0212543a1b1f376c9c62c1d365
8+
refs/heads/try2: 7580ef902eaaf402219b47c3f92019c0644086db
99
refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
1010
refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
1111
refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/src/libcollections/dlist.rs

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
use core::prelude::*;
2525

2626
use alloc::owned::Box;
27+
use core::fmt;
2728
use core::iter;
2829
use core::mem;
2930
use core::ptr;
@@ -608,6 +609,19 @@ impl<A: Clone> Clone for DList<A> {
608609
}
609610
}
610611

612+
impl<A: fmt::Show> fmt::Show for DList<A> {
613+
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
614+
try!(write!(f, "["));
615+
616+
for (i, e) in self.iter().enumerate() {
617+
if i != 0 { try!(write!(f, ", ")); }
618+
try!(write!(f, "{}", *e));
619+
}
620+
621+
write!(f, "]")
622+
}
623+
}
624+
611625
#[cfg(test)]
612626
mod tests {
613627
use std::prelude::*;
@@ -1027,6 +1041,17 @@ mod tests {
10271041
}
10281042
}
10291043

1044+
#[test]
1045+
fn test_show() {
1046+
let list: DList<int> = range(0, 10).collect();
1047+
assert!(list.to_str().as_slice() == "[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]");
1048+
1049+
let list: DList<&str> = vec!["just", "one", "test", "more"].iter()
1050+
.map(|&s| s)
1051+
.collect();
1052+
assert!(list.to_str().as_slice() == "[just, one, test, more]");
1053+
}
1054+
10301055
#[cfg(test)]
10311056
fn fuzz_test(sz: int) {
10321057
let mut m: DList<int> = DList::new();

branches/try2/src/libcollections/smallintmap.rs

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717

1818
use core::prelude::*;
1919

20+
use core::fmt;
2021
use core::iter::{Enumerate, FilterMap};
2122
use core::mem::replace;
2223

@@ -176,6 +177,18 @@ impl<V:Clone> SmallIntMap<V> {
176177
}
177178
}
178179

180+
impl<V: fmt::Show> fmt::Show for SmallIntMap<V> {
181+
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
182+
try!(write!(f, r"\{"));
183+
184+
for (i, (k, v)) in self.iter().enumerate() {
185+
if i != 0 { try!(write!(f, ", ")); }
186+
try!(write!(f, "{}: {}", k, *v));
187+
}
188+
189+
write!(f, r"\}")
190+
}
191+
}
179192

180193
macro_rules! iterator {
181194
(impl $name:ident -> $elem:ty, $getter:ident) => {
@@ -461,6 +474,20 @@ mod test_map {
461474
assert!(called);
462475
m.insert(2, box 1);
463476
}
477+
478+
#[test]
479+
fn test_show() {
480+
let mut map = SmallIntMap::new();
481+
let empty = SmallIntMap::<int>::new();
482+
483+
map.insert(1, 2);
484+
map.insert(3, 4);
485+
486+
let map_str = map.to_str();
487+
let map_str = map_str.as_slice();
488+
assert!(map_str == "{1: 2, 3: 4}" || map_str == "{3: 4, 1: 2}");
489+
assert_eq!(format!("{}", empty), "{}".to_string());
490+
}
464491
}
465492

466493
#[cfg(test)]

branches/try2/src/libcore/cmp.rs

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,14 @@
3939
4040
/// Trait for values that can be compared for equality and inequality.
4141
///
42-
/// This trait allows partial equality, where types can be unordered instead of
43-
/// strictly equal or unequal. For example, with the built-in floating-point
44-
/// types `a == b` and `a != b` will both evaluate to false if either `a` or
45-
/// `b` is NaN (cf. IEEE 754-2008 section 5.11).
42+
/// This trait allows for partial equality, for types that do not have an
43+
/// equivalence relation. For example, in floating point numbers `NaN != NaN`,
44+
/// so floating point types implement `PartialEq` but not `Eq`.
4645
///
47-
/// PartialEq only requires the `eq` method to be implemented; `ne` is its negation by
48-
/// default.
46+
/// PartialEq only requires the `eq` method to be implemented; `ne` is defined
47+
/// in terms of it by default. Any manual implementation of `ne` *must* respect
48+
/// the rule that `eq` is a strict inverse of `ne`; that is, `!(a == b)` if and
49+
/// only if `a != b`.
4950
///
5051
/// Eventually, this will be implemented by default for types that implement
5152
/// `Eq`.
@@ -147,9 +148,10 @@ pub fn lexical_ordering(o1: Ordering, o2: Ordering) -> Ordering {
147148
/// PartialOrd only requires implementation of the `lt` method,
148149
/// with the others generated from default implementations.
149150
///
150-
/// However it remains possible to implement the others separately,
151-
/// for compatibility with floating-point NaN semantics
152-
/// (cf. IEEE 754-2008 section 5.11).
151+
/// However it remains possible to implement the others separately for types
152+
/// which do not have a total order. For example, for floating point numbers,
153+
/// `NaN < 0 == false` and `NaN >= 0 == false` (cf. IEEE 754-2008 section
154+
/// 5.11).
153155
#[lang="ord"]
154156
pub trait PartialOrd: PartialEq {
155157
/// This method tests less than (for `self` and `other`) and is used by the `<` operator.

branches/try2/src/librustdoc/html/static/playpen.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
a.attr('target', '_blank');
2323
$(this).append(a);
2424
}, function() {
25-
$(this).find('a').remove();
25+
$(this).find('a.test-arrow').remove();
2626
});
2727
}
2828
}());

branches/try2/src/libsyntax/ext/quote.rs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,12 @@ pub mod rt {
135135
}
136136
}
137137

138+
impl ToSource for ast::Arg {
139+
fn to_source(&self) -> String {
140+
pprust::arg_to_str(self)
141+
}
142+
}
143+
138144
impl<'a> ToSource for &'a str {
139145
fn to_source(&self) -> String {
140146
let lit = dummy_spanned(ast::LitStr(
@@ -264,6 +270,7 @@ pub mod rt {
264270
impl_to_tokens!(Generics)
265271
impl_to_tokens!(@ast::Expr)
266272
impl_to_tokens!(ast::Block)
273+
impl_to_tokens!(ast::Arg)
267274
impl_to_tokens_self!(&'a str)
268275
impl_to_tokens!(())
269276
impl_to_tokens!(char)

branches/try2/src/libsyntax/print/pprust.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -242,6 +242,10 @@ pub fn variant_to_str(var: &ast::Variant) -> String {
242242
to_str(|s| s.print_variant(var))
243243
}
244244

245+
pub fn arg_to_str(arg: &ast::Arg) -> String {
246+
to_str(|s| s.print_arg(arg))
247+
}
248+
245249
pub fn visibility_qualified(vis: ast::Visibility, s: &str) -> String {
246250
match vis {
247251
ast::Public => format!("pub {}", s).to_string(),

branches/try2/src/test/compile-fail/borrowck-field-sensitivity.rs

Lines changed: 38 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,13 @@
1010

1111
struct A { a: int, b: Box<int> }
1212

13-
fn borrow<T>(_: &T) { }
14-
15-
fn use_after_move() {
13+
fn deref_after_move() {
1614
let x = A { a: 1, b: box 2 };
1715
drop(x.b);
1816
drop(*x.b); //~ ERROR use of partially moved value: `*x.b`
1917
}
2018

21-
fn use_after_fu_move() {
19+
fn deref_after_fu_move() {
2220
let x = A { a: 1, b: box 2 };
2321
let y = A { a: 3, .. x };
2422
drop(*x.b); //~ ERROR use of partially moved value: `*x.b`
@@ -27,35 +25,37 @@ fn use_after_fu_move() {
2725
fn borrow_after_move() {
2826
let x = A { a: 1, b: box 2 };
2927
drop(x.b);
30-
borrow(&x.b); //~ ERROR use of moved value: `x.b`
28+
let p = &x.b; //~ ERROR use of moved value: `x.b`
29+
drop(**p);
3130
}
3231

3332
fn borrow_after_fu_move() {
3433
let x = A { a: 1, b: box 2 };
3534
let _y = A { a: 3, .. x };
36-
borrow(&x.b); //~ ERROR use of moved value: `x.b`
35+
let p = &x.b; //~ ERROR use of moved value: `x.b`
36+
drop(**p);
3737
}
3838

3939
fn move_after_borrow() {
4040
let x = A { a: 1, b: box 2 };
41-
let y = &x.b;
41+
let p = &x.b;
4242
drop(x.b); //~ ERROR cannot move out of `x.b` because it is borrowed
43-
borrow(&*y);
43+
drop(**p);
4444
}
4545

4646
fn fu_move_after_borrow() {
4747
let x = A { a: 1, b: box 2 };
48-
let y = &x.b;
49-
let _z = A { a: 3, .. x }; //~ ERROR cannot move out of `x.b` because it is borrowed
50-
borrow(&*y);
48+
let p = &x.b;
49+
let _y = A { a: 3, .. x }; //~ ERROR cannot move out of `x.b` because it is borrowed
50+
drop(**p);
5151
}
5252

5353
fn mut_borrow_after_mut_borrow() {
5454
let mut x = A { a: 1, b: box 2 };
55-
let y = &mut x.a;
56-
let z = &mut x.a; //~ ERROR cannot borrow `x.a` as mutable more than once at a time
57-
drop(*y);
58-
drop(*z);
55+
let p = &mut x.a;
56+
let q = &mut x.a; //~ ERROR cannot borrow `x.a` as mutable more than once at a time
57+
drop(*p);
58+
drop(*q);
5959
}
6060

6161
fn move_after_move() {
@@ -84,7 +84,21 @@ fn fu_move_after_fu_move() {
8484

8585
// The following functions aren't yet accepted, but they should be.
8686

87-
fn use_after_field_assign_after_uninit() {
87+
fn move_after_borrow_correct() {
88+
let x = A { a: 1, b: box 2 };
89+
let p = &x.a;
90+
drop(x.b); //~ ERROR cannot move out of `x.b` because it is borrowed
91+
drop(*p);
92+
}
93+
94+
fn fu_move_after_borrow_correct() {
95+
let x = A { a: 1, b: box 2 };
96+
let p = &x.a;
97+
let _y = A { a: 3, .. x }; //~ ERROR cannot move out of `x.b` because it is borrowed
98+
drop(*p);
99+
}
100+
101+
fn copy_after_field_assign_after_uninit() {
88102
let mut x: A;
89103
x.a = 1;
90104
drop(x.a); //~ ERROR use of possibly uninitialized variable: `x.a`
@@ -93,7 +107,8 @@ fn use_after_field_assign_after_uninit() {
93107
fn borrow_after_field_assign_after_uninit() {
94108
let mut x: A;
95109
x.a = 1;
96-
borrow(&x.a); //~ ERROR use of possibly uninitialized variable: `x.a`
110+
let p = &x.a; //~ ERROR use of possibly uninitialized variable: `x.a`
111+
drop(*p);
97112
}
98113

99114
fn move_after_field_assign_after_uninit() {
@@ -103,8 +118,8 @@ fn move_after_field_assign_after_uninit() {
103118
}
104119

105120
fn main() {
106-
use_after_move();
107-
use_after_fu_move();
121+
deref_after_move();
122+
deref_after_fu_move();
108123

109124
borrow_after_move();
110125
borrow_after_fu_move();
@@ -117,7 +132,10 @@ fn main() {
117132
fu_move_after_move();
118133
fu_move_after_fu_move();
119134

120-
use_after_field_assign_after_uninit();
135+
move_after_borrow_correct();
136+
fu_move_after_borrow_correct();
137+
138+
copy_after_field_assign_after_uninit();
121139
borrow_after_field_assign_after_uninit();
122140
move_after_field_assign_after_uninit();
123141
}

0 commit comments

Comments
 (0)