Re: [PATCH][RFC] svnadmin load --ignore-dates

C. Michael Pilato wrote:

> Mostly looking
> for general feedback on the idea of the "sounds useful if it works" or
> "you *really* don't want to go there" variety, but deeper review is of
> course welcome, too.

It seems a reasonable enhancement to me. Simple and to the point.

(A couple of style nits:

-  is_date = strcmp(name, SVN_PROP_REVISION_DATE) ? FALSE : TRUE;
+  is_date = strcmp(name, SVN_PROP_REVISION_DATE) == 0;

and you unnecessarily moved the block of code that stores the loaded date stamp into the baton.)

- Julian
Received on 2014-02-05 11:46:25 CET