Do not require Default for collect_result_into() target

neithernut · neithernut · commit 091842951134 · 2018-04-23T20:36:04.000+02:00
While it is required for `ResultIterExt::collect_result_into()`, we do
not need `Default` for `collect_result_into()`. The target already
exists.

The requirement was likely introduced because of C&amp;P.
diff --git a/lib/src/utils.rs b/lib/src/utils.rs
@@ -26,14 +26,14 @@ pub trait ResultIterExt<I, E> : Sized {
     }
 
     fn collect_result_into<T>(self, target: &mut T) -> RResult<(), E>
-        where T: Extend<I> + Default;
+        where T: Extend<I>;
 }
 
 impl<I, E, J> ResultIterExt<I, E> for J
     where J: Iterator<Item = RResult<I, E>>
 {
     fn collect_result_into<T>(self, target: &mut T) -> RResult<(), E>
-        where T: Extend<I> + Default
+        where T: Extend<I>
     {
         for item in self {
             target.extend(Some(item?));

Original file line number	Diff line number	Diff line change
`@@ -26,14 +26,14 @@ pub trait ResultIterExt<I, E> : Sized {`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`fn collect_result_into<T>(self, target: &mut T) -> RResult<(), E>`
`29`		`- where T: Extend<I> + Default;`
	`29`	`+ where T: Extend<I>;`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`impl<I, E, J> ResultIterExt<I, E> for J`
`33`	`33`	`where J: Iterator<Item = RResult<I, E>>`
`34`	`34`	`{`
`35`	`35`	`fn collect_result_into<T>(self, target: &mut T) -> RResult<(), E>`
`36`		`- where T: Extend<I> + Default`
	`36`	`+ where T: Extend<I>`
`37`	`37`	`{`
`38`	`38`	`for item in self {`
`39`	`39`	`target.extend(Some(item?));`