removed asIterator. Its semantics were very dodgy

src/generic.scala

@@ -19,7 +19,7 @@ trait Generic extends CoreProtocol{
   abstract class LengthEncoded[S <: Collection[T], T](implicit binT : Format[T]) extends Format[S]{
     def build(size : Int, ts : Iterator[T]) : S;
 
-    def reads(in : Input) = { val size = read[Int](in); build(size, asIterator[T](in).take(size)) }
+    def reads(in : Input) = { val size = read[Int](in); build(size, (0 until size).map(i => read[T](in)).elements) }
     def writes(out : Output, ts : S) = { write(out, ts.size); ts.foreach(write(out, _)); }
   }
 

src/operations.scala

@@ -11,19 +11,6 @@ object Operations{
   def write[T](out : Output, value : T)(implicit writer : Writes[T]) = writer.writes(out, value);
 
 
-  /**
-   * Returns an iterator that iterates by reading from this input.
-   * In order to ensure proper laziness properties (and not reading more
-   * data than is strictly neccessary) this will always return true 
-   * from hasNext but may throw an EOFException on an unexpected end
-   * of stream.
-   */
-  def asIterator[S](input : Input)(implicit bin : Reads[S]) = new Iterator[S]{
-    def hasNext = true;
-    def next = read[S](input);
-  }
-
-
   /**
    * Get the serialized value of this class as a byte array.
    */