David Barr wrote: > Added 1 new dump header, SVN-fs-dump-format-version. > Added 6 new node headers: Style: using the imperative mood, present tense would make this fit better with other log entries. > --- a/vcs-svn/svndump.c > +++ b/vcs-svn/svndump.c > @@ -27,6 +27,9 @@ > #define LENGTH_UNKNOWN (~0) > #define DATE_RFC2822_LEN 31 > > +#define MD5_HEX_LENGTH 32 > +#define SHA1_HEX_LENGTH 40 I was surprised to find this is not in any existing header. Looks good to me. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html