On Sat, 2011-08-06, Mark Phippard wrote: 
>         The attached Java program shows a bug in 1.7 when calling the
>         getMergeinfoLog API.  If the discoverChangedPaths boolean is
>         set to true we do not get back the expected results.  When it
>         is false, the API works properly. 
[...] 
> It still boils down to the discoverChangedPaths boolean breaks the
> functionality.  I have traced the source code into svn_client and it
> all seems OK to me.  There is nothing obvious about this particular
> parameter that seems wrong.  Maybe the code that calls svn_client_log5
> is not properly setup to handle the additional callbacks for the
> changed paths?  That is all I can think of.

The other day I noticed that the 'svn mergeinfo' command passes
'discover_changed_paths = TRUE' to svn_client_mergeinfo_log(), although
it doesn't need this information.  Expecting to improve performance, I
changed the parameter to FALSE and found that the result was actually
slower.  Looking further now, I see that the result is different.

Example: svn mergeinfo --show-revs=eligible
^/subversion/branches/1.7.x@1154862 ^/subversion/trunk@1154862

  with svn_client_mergeinfo_log(discover_changed_paths=TRUE):
    r1150779
    r1151037
    ...
    r1154720
    r1154734

  with svn_client_mergeinfo_log(discover_changed_paths=TRUE):
    r1148717
    r1148828
    ...
    r1150779
    r1151037
    ...
    r1154720
    r1154734

- Julian



Reply via email to