Re: Issue 1241 - svnlook diff not printing property diffs

John Szakmeister <john@szakmeister.net> writes:

> Actually, I've been dying for some feedback about the patch.

...And I've been meaning to review it. I think I'll do so now. Ah,
yes -- this is the one where you copied a bunch of functions from one
module to another. I remember being bummed that we didn't just find a
nice place to expose the original functions once, and use them in both
places. Oh, and I see you overlooked (or ignored) the fact that
svnlook/main.c uses the fn-space-paren style.

But tonight, I'm feeling good, so I'll just rework your patch the way
I would have liked to see it. You get a free one, on me. :-)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 8 08:07:04 2003