We can add test to check no one uses iterators but rather only
enumerators... this should not be too hard..


On Sun, Oct 27, 2013 at 4:43 PM, yarondbb <notifications@github.com> wrote:

> In lib/content_data/content_data.rb:
>
> >        }
> >        file.write("#{@instances_info.length}\n")
> > -      each_instance { |checksum, size, _, instance_mod_time, server, path|
> > -        file.write("#{checksum},#{size},#{server},#{path},#{instance_mod_time}\n")
> > +      contents_enum = @contents_info.each_key
> > +      instances_chunks = @instances_info.length / 5000 + 1
> > +      instances_chunks.times { |index|
> > +        to_file_instances_chunk(file,contents_enum,5000)
> > +        GC.start
>
> Yes. GC takes time.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bbfsdev/bbfs/pull/231/files#r7236742>
> .
>