Re: [Patch] JavaHL add an implementation of BlameCallback and change code to use it

Mark Phippard wrote:
> I have attached a patch that does what we discussed on IRC. It adds an
> implementation of BlameCallback to JavaHL and then changes SVNClient and
> BasicTests to use it. This gets rid of the duplicate internal
> BlameReceiver
> class.
>
> I am not setup to build Subversion at the moment, so I cannot run the test
> suite. The code formatting and JavaDoc of the new class also might not be
> up to standards. I can fix it up, I just wanted to see if I am on the
> right
> track with the patch.

Aside from the large quantity of tab characters, things look okay. I
applied the patch and ran the test suite, with no problems. Does the
BlameLine class need to be public, or can it be protected? It is only
used internally, and only gets returned from the protected
getBlameLine() method.

-Hyrum