[MERGE][RFC] Add simple revision serializer based on RIO.

Jelmer Vernooij jelmer at vernstok.nl
Wed May 6 20:55:05 BST 2009


On Wed, May 06, 2009 at 04:31:58PM +0200, Jelmer Vernooij wrote:
> On Wed, May 06, 2009 at 09:19:45AM -0400, Ian Clatworthy wrote:
> > Ian Clatworthy has voted comment.
> > Status is now: Semi-approved
> > Comment:
> > A performance improvement would be one of the main reasons for including  
> > this or otherwise. I'd like to see this patch resubmitted so that this  
> > serialiser is used in a proposed new development format. (At first  
> > glance, it looks like your changing the behaviour of an existing  
> > format?) I can then more easily benchmark its impact, report the results  
> > and vote accordingly.

> Another improvement is that it makes the escape_commit_message hack

it makes that hack *unnecessary* is what I was trying to say :-)

> that we currently have in the commit code and in foreign branch
> implementations. This hack is required since the current format does
> not support XML-unescapable characters in any of these fields.


Cheers,

Jelmer



More information about the bazaar mailing list