Re: [PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

2010-11-16 Thread Stefan Fuhrmann
On 15.11.2010 23:22, Gavin Beau Baumanis wrote: Ping. This has received no further comments. Gavin "Beau" Baumanis E: gav...@thespidernet.com On 31/10/2010, at 12:42 PM, Daniel Shahaf wrote: Is there a reason not to apply this "let's not reinvent the wheel" patch? [[[ Index: subversion/li

Re: [PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

2010-11-16 Thread Stefan Fuhrmann
On 17.11.2010 00:11, Daniel Shahaf wrote: Hyrum K. Wright wrote on Tue, Nov 16, 2010 at 13:02:04 -0600: Since Daniel is a committer, I suspect he will track this change (relieving you of the burden of doing so). That's correct. -Hyrum I'm not decided about that patch, yet. id_unparse is rel

Re: [PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

2010-11-16 Thread Daniel Shahaf
Hyrum K. Wright wrote on Tue, Nov 16, 2010 at 13:02:04 -0600: > Since Daniel is a committer, I suspect he will track this change > (relieving you of the burden of doing so). > That's correct. > -Hyrum > > On Mon, Nov 15, 2010 at 4:22 PM, Gavin Beau Baumanis > wrote: > > Ping. This has received

Re: [PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

2010-11-16 Thread Hyrum K. Wright
Since Daniel is a committer, I suspect he will track this change (relieving you of the burden of doing so). -Hyrum On Mon, Nov 15, 2010 at 4:22 PM, Gavin Beau Baumanis wrote: > Ping. This has received no further comments. > > > > Gavin "Beau" Baumanis > E: gav...@thespidernet.com > > > On 31/10/

Re: [PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

2010-11-15 Thread Gavin Beau Baumanis
Ping. This has received no further comments. Gavin "Beau" Baumanis E: gav...@thespidernet.com On 31/10/2010, at 12:42 PM, Daniel Shahaf wrote: > Is there a reason not to apply this "let's not reinvent the wheel" patch? > > [[[ > Index: subversion/libsvn_fs_fs/fs_fs.c > ==

[PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

2010-10-30 Thread Daniel Shahaf
Is there a reason not to apply this "let's not reinvent the wheel" patch? [[[ Index: subversion/libsvn_fs_fs/fs_fs.c === --- subversion/libsvn_fs_fs/fs_fs.c (revision 1029231) +++ subversion/libsvn_fs_fs/fs_fs.c (working copy)