[darcs-users] darcs patch: Split Cache mostly out of Darsc/Repository/Prefs into its own file (take 2)

Jason Dagit dagit at codersbase.com
Wed Aug 13 16:41:10 UTC 2008


On Wed, Aug 13, 2008 at 3:02 AM, Nathaniel W Filardo <nwf at cs.jhu.edu> wrote:

> On Sat, Aug 09, 2008 at 03:53:40AM -0400, Nathaniel W Filardo wrote:
> > Split the material dealing with hashed repository formats and caches
> mostly
> > out of Prefs (into InternalTypes and HashedIO).  This restores Prefs to
> > being mostly _darcs/prefs/* handling code.  Compiles and passes tests,
> but
> > not sure it's the maximally right answer.
>
> This is a second attempt at this patch, which instead of moving things into
> InternalTypes and HashedIO, moves the Cache logic out to
> Darcs/Repository/Cache.  This is much less involved than the last patch,
> being mostly a move and scattered updates to imports.
>
> The one notable change it does make (as a second patch in the bundle) is to
> export the cache hash function in hopes of it being useful for issue861.
>  If
> desired, I can submit a version without.


At a glance these patches look fine to me, but I didn't test them out.  If I
understand correctly, the point of these refactors are make it easier to
solve some issues in the bug tracker?

Thanks,
Jason
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.osuosl.org/pipermail/darcs-users/attachments/20080813/68a23ee4/attachment.htm 


More information about the darcs-users mailing list