]> git.lizzy.rs Git - rust.git/commitdiff
fix fallout
authorJorge Aparicio <japaricious@gmail.com>
Sun, 11 Jan 2015 02:50:07 +0000 (21:50 -0500)
committerJorge Aparicio <japaricious@gmail.com>
Fri, 30 Jan 2015 15:37:44 +0000 (10:37 -0500)
31 files changed:
src/doc/trpl/unsafe.md
src/libcollections/bit.rs
src/libcollections/btree/node.rs
src/libcollections/lib.rs
src/libcollections/ring_buf.rs
src/libcollections/slice.rs
src/libcollections/str.rs
src/libcollections/vec.rs
src/libcore/iter.rs
src/libcore/lib.rs
src/libcore/num/mod.rs
src/librand/lib.rs
src/librustc/lib.rs
src/librustc/middle/dataflow.rs
src/librustc_trans/lib.rs
src/librustc_typeck/lib.rs
src/librustdoc/lib.rs
src/libserialize/collection_impls.rs
src/libstd/ascii.rs
src/libstd/collections/hash/table.rs
src/libstd/lib.rs
src/libstd/path/windows.rs
src/libstd/rand/os.rs
src/libstd/sync/mpsc/mod.rs
src/libstd/sync/mpsc/mpsc_queue.rs
src/libsyntax/fold.rs
src/libsyntax/lib.rs
src/libtest/lib.rs
src/libunicode/lib.rs
src/test/bench/shootout-mandelbrot.rs
src/test/bench/shootout-meteor.rs

index 3acd1eefe89d06f1b7a8578eaa76c1ff7b4b13b4..2bd86fa987f4be962bd3ec3086dd5ee2b8cf5b2e 100644 (file)
@@ -576,6 +576,10 @@ extern fn panic_fmt(args: &core::fmt::Arguments,
 #[lang = "eh_personality"] extern fn eh_personality() {}
 # #[start] fn start(argc: isize, argv: *const *const u8) -> isize { 0 }
 # fn main() {}
+# mod std {  // for-loops
+#     pub use core::iter;
+#     pub use core::option;
+# }
 ```
 
 Note that there is one extra lang item here which differs from the examples
index d676cfca9294e6ab2a0f07a03c94aeed28a36055..99172dcdef3cca42626dcce754f2e84548342073 100644 (file)
 
 fn reverse_bits(byte: u8) -> u8 {
     let mut result = 0;
-    for i in 0..u8::BITS {
+    // FIXME(#21245) use a for loop
+    let mut iter = 0..u8::BITS;
+    while let Some(i) = iter.next() {
         result |= ((byte >> i) & 1) << (u8::BITS - 1 - i);
     }
     result
index ea167348a644cf7f4ebdc644b526bbfab87b181a..8d6f06b25c5be42512197325e74381087784be10 100644 (file)
@@ -271,7 +271,7 @@ fn next_back(&mut self) -> Option<T> {
 #[unsafe_destructor]
 impl<T> Drop for RawItems<T> {
     fn drop(&mut self) {
-        for _ in *self {}
+        for _ in self.by_ref() {}
     }
 }
 
@@ -1374,9 +1374,9 @@ impl<K, V> Drop for MoveTraversalImpl<K, V> {
     fn drop(&mut self) {
         // We need to cleanup the stored values manually, as the RawItems destructor would run
         // after our deallocation.
-        for _ in self.keys {}
-        for _ in self.vals {}
-        for _ in self.edges {}
+        for _ in self.keys.by_ref() {}
+        for _ in self.vals.by_ref() {}
+        for _ in self.edges.by_ref() {}
 
         let (alignment, size) =
                 calculate_allocation_generic::<K, V>(self.capacity, self.is_leaf);
index 82b92d26d28bc51d1f1f003a6c7c3b7f18a8f2bd..f28262dc0fea746ab5c06c0ac394810414428afd 100644 (file)
@@ -34,6 +34,8 @@
 #![feature(unicode)]
 #![feature(hash)]
 #![cfg_attr(test, feature(test))]
+// NOTE(stage0): remove after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 #[macro_use]
 extern crate core;
@@ -114,6 +116,8 @@ mod std {
     pub use core::marker;   // derive(Copy)
     pub use core::hash;     // derive(Hash)
     pub use core::ops;      // RangeFull
+    // for-loops
+    pub use core::iter;
 }
 
 #[cfg(test)]
index 5b5e3be12e35fb5688ca3d2d03aeb2e7cf863c99..a9b31da8250e12850c882f473d6f55ebb97f2232 100644 (file)
@@ -1510,7 +1510,7 @@ pub struct Drain<'a, T: 'a> {
 #[stable(feature = "rust1", since = "1.0.0")]
 impl<'a, T: 'a> Drop for Drain<'a, T> {
     fn drop(&mut self) {
-        for _ in *self {}
+        for _ in self.by_ref() {}
         self.inner.head = 0;
         self.inner.tail = 0;
     }
@@ -1793,7 +1793,9 @@ fn bench_new(b: &mut test::Bencher) {
     fn bench_push_back_100(b: &mut test::Bencher) {
         let mut deq = RingBuf::with_capacity(101);
         b.iter(|| {
-            for i in 0i..100 {
+            // FIXME(#21245) use a for loop
+            let mut iter = 0i..100;
+            while let Some(i) = iter.next() {
                 deq.push_back(i);
             }
             deq.head = 0;
@@ -1805,7 +1807,9 @@ fn bench_push_back_100(b: &mut test::Bencher) {
     fn bench_push_front_100(b: &mut test::Bencher) {
         let mut deq = RingBuf::with_capacity(101);
         b.iter(|| {
-            for i in 0i..100 {
+            // FIXME(#21245) use a for loop
+            let mut iter = 0i..100;
+            while let Some(i) = iter.next() {
                 deq.push_front(i);
             }
             deq.head = 0;
index 56d969b89466cdb21cf13a82fa0e65d57e10bdb1..57ee4704a9ee099b376bd791ba53be8057a202bb 100644 (file)
@@ -1958,7 +1958,7 @@ fn test_permutations() {
             let mut amt = 0;
             let mut it = v.permutations();
             let (min_size, max_opt) = it.size_hint();
-            for _perm in it {
+            for _perm in it.by_ref() {
                 amt += 1;
             }
             assert_eq!(amt, it.swaps.swaps_made);
index 35591a5e9effb049a22b73ebff18e6a65072dc90..bbed943fc70229c2978d8191a94347903eaf5330 100644 (file)
@@ -279,7 +279,7 @@ fn next(&mut self) -> Option<char> {
         loop {
             match self.state {
                 Composing => {
-                    for ch in self.iter {
+                    for ch in self.iter.by_ref() {
                         let ch_class = unicode::char::canonical_combining_class(ch);
                         if self.composee.is_none() {
                             if ch_class != 0 {
index ac6d7936f280840cf07aec83853e86ffaf0db212..4ea8267135f0e18796c1cf8307fe68195a65955d 100644 (file)
@@ -1567,7 +1567,9 @@ fn drop(&mut self) {
         // zeroed (when moving out, because of #[unsafe_no_drop_flag]).
         if self.cap != 0 {
             unsafe {
-                for x in self.iter() {
+                // FIXME(#21245) use a for loop
+                let mut iter = self.iter();
+                while let Some(x) = iter.next() {
                     ptr::read(x);
                 }
                 dealloc(*self.ptr, self.cap)
@@ -1648,7 +1650,7 @@ impl<T> IntoIter<T> {
     #[unstable(feature = "collections")]
     pub fn into_inner(mut self) -> Vec<T> {
         unsafe {
-            for _x in self { }
+            for _x in self.by_ref() { }
             let IntoIter { allocation, cap, ptr: _ptr, end: _end } = self;
             mem::forget(self);
             Vec { ptr: NonZero::new(allocation), cap: cap, len: 0 }
@@ -1726,7 +1728,7 @@ impl<T> Drop for IntoIter<T> {
     fn drop(&mut self) {
         // destroy the remaining elements
         if self.cap != 0 {
-            for _x in *self {}
+            for _x in self.by_ref() {}
             unsafe {
                 dealloc(self.allocation, self.cap);
             }
@@ -1816,7 +1818,7 @@ fn drop(&mut self) {
         // so we can use #[unsafe_no_drop_flag].
 
         // destroy the remaining elements
-        for _x in *self {}
+        for _x in self.by_ref() {}
     }
 }
 
index f9c6e0758e6e87b18403a7f870ddf7d63ca6d59f..7847ed13ca9f86e73373610fef37b61da16da93a 100644 (file)
@@ -195,7 +195,7 @@ fn last(mut self) -> Option<Self::Item> {
     #[inline]
     #[stable(feature = "rust1", since = "1.0.0")]
     fn nth(&mut self, mut n: usize) -> Option<Self::Item> {
-        for x in *self {
+        for x in self.by_ref() {
             if n == 0 { return Some(x) }
             n -= 1;
         }
@@ -492,7 +492,7 @@ fn flat_map<B, U, F>(self, f: F) -> FlatMap<Self::Item, B, Self, U, F> where
     /// fn process<U: Iterator<Item=isize>>(it: U) -> isize {
     ///     let mut it = it.fuse();
     ///     let mut sum = 0;
-    ///     for x in it {
+    ///     for x in it.by_ref() {
     ///         if x > 5 {
     ///             break;
     ///         }
@@ -660,7 +660,7 @@ fn all<F>(mut self, mut f: F) -> bool where F: FnMut(Self::Item) -> bool {
     #[inline]
     #[stable(feature = "rust1", since = "1.0.0")]
     fn any<F>(&mut self, mut f: F) -> bool where F: FnMut(Self::Item) -> bool {
-        for x in *self { if f(x) { return true; } }
+        for x in self.by_ref() { if f(x) { return true; } }
         false
     }
 
@@ -680,7 +680,7 @@ fn any<F>(&mut self, mut f: F) -> bool where F: FnMut(Self::Item) -> bool {
     fn find<P>(&mut self, mut predicate: P) -> Option<Self::Item> where
         P: FnMut(&Self::Item) -> bool,
     {
-        for x in *self {
+        for x in self.by_ref() {
             if predicate(&x) { return Some(x) }
         }
         None
@@ -703,7 +703,7 @@ fn position<P>(&mut self, mut predicate: P) -> Option<usize> where
         P: FnMut(Self::Item) -> bool,
     {
         let mut i = 0;
-        for x in *self {
+        for x in self.by_ref() {
             if predicate(x) {
                 return Some(i);
             }
@@ -1664,7 +1664,7 @@ impl<A, I, P> Iterator for Filter<A, I, P> where I: Iterator<Item=A>, P: FnMut(&
 
     #[inline]
     fn next(&mut self) -> Option<A> {
-        for x in self.iter {
+        for x in self.iter.by_ref() {
             if (self.predicate)(&x) {
                 return Some(x);
             } else {
@@ -1728,7 +1728,7 @@ impl<A, B, I, F> Iterator for FilterMap<A, B, I, F> where
 
     #[inline]
     fn next(&mut self) -> Option<B> {
-        for x in self.iter {
+        for x in self.iter.by_ref() {
             match (self.f)(x) {
                 Some(y) => return Some(y),
                 None => ()
@@ -1914,7 +1914,7 @@ impl<A, I, P> Iterator for SkipWhile<A, I, P> where I: Iterator<Item=A>, P: FnMu
 
     #[inline]
     fn next(&mut self) -> Option<A> {
-        for x in self.iter {
+        for x in self.iter.by_ref() {
             if self.flag || !(self.predicate)(&x) {
                 self.flag = true;
                 return Some(x);
@@ -2207,7 +2207,7 @@ impl<A, B, I, U, F> Iterator for FlatMap<A, B, I, U, F> where
     fn next(&mut self) -> Option<B> {
         loop {
             for inner in self.frontiter.iter_mut() {
-                for x in *inner {
+                for x in inner.by_ref() {
                     return Some(x)
                 }
             }
index 1032c56fa22ad50f05b8bdab87cc8038fb23dda9..353d4252dfb05c296bcf7049710d97014adab999 100644 (file)
@@ -65,6 +65,8 @@
 #![allow(unknown_features)] #![feature(int_uint)]
 #![feature(on_unimplemented)]
 #![deny(missing_docs)]
+// NOTE(stage0) remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 #[macro_use]
 mod macros;
@@ -158,4 +160,6 @@ mod std {
     pub use marker;
     pub use ops;
     pub use option;
+    // for-loops
+    pub use iter;
 }
index dd9cc553c7c8f34b6a80aff46537fbf2956d89a2..1be7a0fb066b5bb8b384fd94841df901deb749bd 100644 (file)
@@ -1524,7 +1524,7 @@ fn from_str_radix(src: &str, radix: uint) -> Option<$T> {
                 let mut exp_info = None::<(char, uint)>;
 
                 // Parse the integer part of the significand
-                for (i, c) in cs {
+                for (i, c) in cs.by_ref() {
                     match c.to_digit(radix) {
                         Some(digit) => {
                             // shift significand one digit left
@@ -1572,7 +1572,7 @@ fn from_str_radix(src: &str, radix: uint) -> Option<$T> {
                 // part of the significand
                 if exp_info.is_none() {
                     let mut power = 1.0;
-                    for (i, c) in cs {
+                    for (i, c) in cs.by_ref() {
                         match c.to_digit(radix) {
                             Some(digit) => {
                                 // Decrease power one order of magnitude
index 762581518505f675b349e5b2696773fa12890438..8c50550a6e84cd3a8c78d8c7be48f55f05ee1977 100644 (file)
@@ -498,6 +498,8 @@ mod std {
     pub use core::{option, fmt}; // panic!()
     pub use core::clone; // derive Clone
     pub use core::marker;
+    // for-loops
+    pub use core::iter;
 }
 
 #[cfg(test)]
index 580e55f78a9e5385504780e42af0a663df5fc7e2..e99b203cce71f2362f1ce2f575624450e690cad0 100644 (file)
@@ -41,6 +41,9 @@
 #![feature(unicode)]
 #![feature(hash)]
 #![cfg_attr(test, feature(test))]
+#![allow(unstable)]
+// NOTE(stage0) remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 extern crate arena;
 extern crate flate;
index 01d42523f3556d6be755185075bf50a61bd75fc9..d5ee0e57b79685a5eaa2a2b492b2302d5986c2f8 100644 (file)
@@ -352,7 +352,9 @@ fn each_bit<F>(&self, words: &[uint], mut f: F) -> bool where
         for (word_index, &word) in words.iter().enumerate() {
             if word != 0 {
                 let base_index = word_index * uint::BITS;
-                for offset in 0u..uint::BITS {
+                // FIXME(#21245) use a for loop
+                let mut iter = 0u..uint::BITS;
+                while let Some(offset) = iter.next() {
                     let bit = 1 << offset;
                     if (word & bit) != 0 {
                         // NB: we round up the total number of bits
index bb026e237df82051b917b20a5b25ba9b1e596c1a..5a2ff926669212bacef36a7fa5a1b4b1e5087afb 100644 (file)
@@ -41,6 +41,9 @@
 #![feature(std_misc)]
 #![feature(unicode)]
 #![feature(hash)]
+#![allow(unstable)]
+// NOTE(stage0) remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 extern crate arena;
 extern crate flate;
index 31c9f9961260a116552ed2fa1b70cb97d05d1f7e..4772fac8bd4593f1fff7a38d77ff51cb67bfba34 100644 (file)
@@ -84,6 +84,9 @@
 #![feature(core)]
 #![feature(rustc_private)]
 #![feature(std_misc)]
+#![allow(unstable)]
+// NOTE(stage0) remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 #[macro_use] extern crate log;
 #[macro_use] extern crate syntax;
index f8ba2dc2a74837683290685bc73b43e4b5d3db3f..974d8b96d32e3ad970b20a8ded359f8c2549df0a 100644 (file)
@@ -32,6 +32,9 @@
 #![feature(test)]
 #![feature(unicode)]
 #![feature(hash)]
+#![allow(unstable)]
+// NOTE(stage0) remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 extern crate arena;
 extern crate getopts;
index 8b39d91ffaeb0f029af4ad0493e2089cf9634b8c..77308d0a66f62bcad0327b23dc72148f5a2a4847 100644 (file)
@@ -148,7 +148,9 @@ impl<
     fn decode<D: Decoder>(d: &mut D) -> Result<EnumSet<T>, D::Error> {
         let bits = try!(d.read_uint());
         let mut set = EnumSet::new();
-        for bit in 0..uint::BITS {
+        // FIXME(#21245) use a for loop
+        let mut iter = 0..uint::BITS;
+        while let Some(bit) = iter.next() {
             if bits & (1 << bit) != 0 {
                 set.insert(CLike::from_uint(1 << bit));
             }
index 9aa38e711e70f3501b1ccd49d5e736a20e46c13e..ffc4c5c6bacebf6c5d4a47db42ee1168339dcb92 100644 (file)
@@ -232,7 +232,10 @@ pub fn escape_default<F>(c: u8, mut f: F) where
         _ => {
             f(b'\\');
             f(b'x');
-            for &offset in [4u, 0u].iter() {
+            // FIXME(#21245) use a for loop
+            let arr = [4u, 0u];
+            let mut iter = arr.iter();
+            while let ::option::Option::Some(&offset) = ::iter::Iterator::next(&mut iter) {
                 match ((c as i32) >> offset) & 0xf {
                     i @ 0 ... 9 => f(b'0' + (i as u8)),
                     i => f(b'a' + (i as u8 - 10)),
index 9e6a45d8bf0b5877fa3ae8159d83d61e8b120cb8..141c51d8363d86de319d49ae3842347fa49747f5 100644 (file)
@@ -15,7 +15,7 @@
 use clone::Clone;
 use cmp;
 use hash::{Hash, Hasher};
-use iter::{Iterator, ExactSizeIterator, count};
+use iter::{Iterator, IteratorExt, ExactSizeIterator, count};
 use marker::{Copy, Sized, self};
 use mem::{min_align_of, size_of};
 use mem;
@@ -921,7 +921,7 @@ fn len(&self) -> usize { self.table.size() }
 #[unsafe_destructor]
 impl<'a, K: 'a, V: 'a> Drop for Drain<'a, K, V> {
     fn drop(&mut self) {
-        for _ in *self {}
+        for _ in self.by_ref() {}
     }
 }
 
index a016ba8fb0cd069853b114773d359d9277af0c72..534ece52ff5ff2e7252c55c45c4b6c17b466a8c3 100644 (file)
 #![feature(rand)]
 #![feature(hash)]
 #![cfg_attr(test, feature(test))]
+#![allow(unstable)]
+// NOTE(stage0): remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 // Don't link to std. We are std.
 #![no_std]
@@ -310,4 +313,6 @@ mod std {
     pub use slice;
 
     pub use boxed; // used for vec![]
+    // for-loops
+    pub use iter;
 }
index 88db27013ac83c43003d7ca101cd8e1acc99c77d..98e0320cd14d21f0c8ee7ae20fbe36b3cb62b5b5 100644 (file)
@@ -557,7 +557,7 @@ fn comp_requires_verbatim(s: &str) -> bool {
                     }
                     (Some(a), Some(_)) => {
                         comps.push("..");
-                        for _ in itb {
+                        for _ in itb.by_ref() {
                             comps.push("..");
                         }
                         comps.push(a);
index 992afb2d10fc487232c6235c0e59bd6359b69b89..05677e1379db081c731385a480a7ca9c128cbd79 100644 (file)
@@ -404,7 +404,9 @@ fn test_os_rng_tasks() {
         }
 
         // start all the tasks
-        for tx in txs.iter() {
+        // FIXME(#21245) use a for loop
+        let mut iter = txs.iter();
+        while let Some(tx) = iter.next() {
             tx.send(()).unwrap();
         }
     }
index 6a43eccbaba4e935945a19caec6a4aecb021d567..b503b92b00f962e17861c30209ebb72fbc8eccb2 100644 (file)
@@ -1476,7 +1476,9 @@ fn test_recv_iter_break() {
 
         let _t = Thread::spawn(move|| {
             let mut count = 0;
-            for x in rx.iter() {
+            // FIXME(#21245) use a for loop
+            let mut iter = rx.iter();
+            while let Some(x) = iter.next() {
                 if count >= 3 {
                     break;
                 } else {
@@ -1940,7 +1942,9 @@ fn test_recv_iter_break() {
 
         let _t = Thread::spawn(move|| {
             let mut count = 0;
-            for x in rx.iter() {
+            // FIXME(#21245) use a for loop
+            let mut iter = rx.iter();
+            while let Some(x) = iter.next() {
                 if count >= 3 {
                     break;
                 } else {
index 53eba131674cf6b45d79ddd084fe9181138eba1b..cc09cfd665c7a098a871a331ef881cb13367c86c 100644 (file)
@@ -185,7 +185,9 @@ fn test() {
             let tx = tx.clone();
             let q = q.clone();
             Thread::spawn(move|| {
-                for i in 0..nmsgs {
+                // FIXME(#21245) use a for loop
+                let mut iter = 0..nmsgs;
+                while let Some(i) = iter.next() {
                     q.push(i);
                 }
                 tx.send(()).unwrap();
index a1362f5382c979b4f3864039181f5e7e2fd5e434..d739e5fe6a03aa70b5f927d6d24fd985792f227c 100644 (file)
@@ -37,10 +37,14 @@ pub trait MoveMap<T> {
 
 impl<T> MoveMap<T> for Vec<T> {
     fn move_map<F>(mut self, mut f: F) -> Vec<T> where F: FnMut(T) -> T {
-        for p in self.iter_mut() {
-            unsafe {
-                // FIXME(#5016) this shouldn't need to zero to be safe.
-                ptr::write(p, f(ptr::read_and_zero(p)));
+        // FIXME(#21245) use a for loop
+        {
+            let mut iter = self.iter_mut();
+            while let Some(p) = iter.next() {
+                unsafe {
+                    // FIXME(#5016) this shouldn't need to zero to be safe.
+                    ptr::write(p, f(ptr::read_and_zero(p)));
+                }
             }
         }
         self
index ff4c7b565cbbe469375cac7706eb8b0bf8882c4a..456c6f73ce1eb9888399cf8b3233866e6affcfc8 100644 (file)
@@ -39,6 +39,9 @@
 #![feature(rustc_private)]
 #![feature(std_misc)]
 #![feature(unicode)]
+#![allow(unstable)]
+// NOTE(stage0) remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 extern crate arena;
 extern crate fmt_macros;
index 8fea6bb539f57331fb821b7bcc077b2fb1d281df..af8d5b86f4bd5a0a8e45e6551af864696e51fbab 100644 (file)
@@ -44,6 +44,9 @@
 #![feature(rustc_private)]
 #![feature(std_misc)]
 #![feature(hash)]
+#![allow(unstable)]
+// NOTE(stage0): remove cfg_attr after a snapshot
+#![cfg_attr(not(stage0), allow(unused_mut))]
 
 extern crate getopts;
 extern crate serialize;
index 02f738c9d29e1c2e50437e2d7252c31b2541e8e0..659d57b729214e65dbcb19b9bc56e661e4c0f130 100644 (file)
@@ -84,4 +84,7 @@ mod std {
     pub use core::cmp;
     pub use core::fmt;
     pub use core::marker;
+    // for-loops
+    pub use core::iter;
+    pub use core::option;
 }
index 4a9c5a91dcf0bb5cbe976946e7782e8f9225c627..e84e6ac5699e4baaaf81ad4b564456f98e1a994e 100644 (file)
@@ -133,7 +133,9 @@ fn mandelbrot<W: old_io::Writer>(w: uint, mut out: W) -> old_io::IoResult<()> {
                 (i + 1) * chunk_size
             };
 
-            for &init_i in vec_init_i[start..end].iter() {
+            // FIXME(#21245) use a for loop
+            let mut iter = vec_init_i[start..end].iter();
+            while let Some(&init_i) = iter.next() {
                 write_line(init_i, init_r_slice, &mut res);
             }
 
@@ -142,7 +144,9 @@ fn mandelbrot<W: old_io::Writer>(w: uint, mut out: W) -> old_io::IoResult<()> {
     }).collect::<Vec<_>>();
 
     try!(writeln!(&mut out as &mut Writer, "P4\n{} {}", w, h));
-    for res in data.into_iter() {
+    // FIXME(#21245) use a for loop
+    let mut iter = data.into_iter();
+    while let Some(res) = iter.next() {
         try!(out.write(res.join().ok().unwrap().as_slice()));
     }
     out.flush()
index e6ef58cba35f83b07f16e5bbaa02e78018a9fd8b..209cc985383bc654c04a97b7b0df4a27fc53013d 100644 (file)
@@ -180,7 +180,9 @@ fn make_masks() -> Vec<Vec<Vec<u64> > > {
 // all unused piece can be placed on the board.
 fn is_board_unfeasible(board: u64, masks: &Vec<Vec<Vec<u64>>>) -> bool {
     let mut coverable = board;
-    for (i, masks_at) in masks.iter().enumerate() {
+    // FIXME(#21245) use a for loop
+    let mut iter = masks.iter().enumerate();
+    while let Some((i, masks_at)) = iter.next() {
         if board & 1 << i != 0 { continue; }
         for (cur_id, pos_masks) in masks_at.iter().enumerate() {
             if board & 1 << (50 + cur_id) != 0 { continue; }
@@ -222,7 +224,9 @@ fn to_vec(raw_sol: &List<u64>) -> Vec<u8> {
     let mut sol = repeat('.' as u8).take(50).collect::<Vec<_>>();
     for &m in raw_sol.iter() {
         let id = '0' as u8 + get_id(m);
-        for i in 0u..50 {
+        // FIXME(#21245) use a for loop
+        let mut iter = 0u..50;
+        while let Some(i) = iter.next() {
             if m & 1 << i != 0 {
                 sol[i] = id;
             }