Hello Luca,

Op woensdag 23 december 2009 12:00 schreef je:
> Have you already applied this? I may have found a bug on it.

I considered applying it tonight, but it raised some more questions on IRC:

  [21:26] <Heffalump> it effectively throws away the existing cache, right?
  [21:26] * Heffalump is a bit dubious about that
  [21:27] <kowey> dubious about that being a good idea?
  [21:31] <tux_rocker> would restructuring the cache be much faster than just   
 
                       rebuilding it?
  [21:32] <idnar> doesn't throwing the cache away potentially lose data?

I think Heffalump was really dubious about throwing away the existing cache 
being a good idea.

Additionally, the patch introduces two compiler warnings:

  [52 of 63] Compiling Darcs.Repository.Cache ( src/Darcs/Repository/Cache.hs,  
       
  dist/build/unit/unit-tmp/Darcs/Repository/Cache.o )                           
 

  src/Darcs/Repository/Cache.hs:114:29:
      Warning: This binding for `f' shadows the existing binding
                 bound at src/Darcs/Repository/Cache.hs:113:39  
               In the definition of `hashedBucket'              

  src/Darcs/Repository/Cache.hs:116:48:
      Warning: This binding for `s' shadows the existing binding
                 bound at src/Darcs/Repository/Cache.hs:113:37  
               In a case alternative

Bye,
Reinier

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to