On 10/25/2012 01:02 PM, Stefan Fuhrmann wrote:
> Hi,
>
> We've managed to widely eliminate the use of deprecated
> functions except for a few cases. One of them is
> svn_delta_path_driver.
>
> The problem is that svn_delta_path_driver2 requires an
> ordered path array as input that none but one of the callers
> can actually provide. We would need to sort the path list
> for all other callers via some copy-n-paste code.
>
> It seems that we can't easily check a given path array
> for being properly ordered inside svn_delta_path_driver2,
> so I'd like to add a flag parameter that tells the function
> whether it needs to sort the array before using it. Alternatively,
> we could add a some __compare_fn_t as a parameter.
> If NULL, assume ordered input, otherwise sort using
> the order defined by the compare function.
>
> Any other ideas?
Would that sorting function ever be anything besides NULL or the wrapper
around svn_path_compare_paths()? I mean, "depth-first sorting" has a pretty
specific definition where driving an Ev1 editor is concerned, so ... seems a
boolean sort-the-paths-for-me-please parameter is the more sensible provision.
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Enterprise Cloud Development
Received on 2012-10-25 19:37:02 CEST