dulwich-users team mailing list archive

[Augie Fackler <durin42@xxxxxxxxx>]

On May 30, 2010, at 9:18 AM, Jelmer Vernooij wrote:

Hi Dave,

On Mon, May 24, 2010 at 11:08:56AM -0700, David Borowitz wrote:

This is quite the grab-bag of changes :). Available as usual at my github

fork:

http://github.com/dborowitz/dulwich


I think Augie has requested that we do code reviews in some place public,

like this mailing list. If others agree I'm happy to oblige by mailing

patches.

That'd be great - mailed patches are a bit easier to comment on for me, and
this
mailing list seems like the most appropriate place.

b9677d0 Add tests for sorted_tree_items and C implementation.

186fb3a Fix memory leak in C implementation of sorted_tree_items.

0994d03 Clean up file headers.

4a42aad Fix copyright in dul-web.

I've merged these, thanks.

e146b26 Move named file initilization to BaseRepo.

Thanks, this makes sense.

We should've really started out by naming BaseRepo Repo and naming Repo
DiskRepo or something, but it seems hard to change that now without breaking
everybody's existing code. :-(

9f08973 Use / as separator for named files on all platforms.

I'm not sure about this change - other than the use of info/excludes we
don't
actually appear to use / in calls to {get,put}_named_file so it seems like
a
silly thing to break the API for.


Is it an API break then? It also means that you can always request
'info/excludes' without respect to the platform you're on.

Alternatively, perhaps we can make get_named_file private/protected? I'd be
less hesitant about making this sort of change if we do that.


-1, it feels like clients of Repo will need to load files within .git and
get_named_file feels like the correct call for this.


f8d06e0 Add a MemoryRepo that stores everything in memory.

Nice!

Cheers,

Jelmer

_______________________________________________
Mailing list: https://launchpad.net/~dulwich-users
Post to     : dulwich-users@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~dulwich-users
More help   : https://help.launchpad.net/ListHelp