Giovanni Bajo wrote:
> If you add this simple class to the option parsing machinery:
 >
 > [...]
>
> I think additional verifications like this better belong to the
> testsuite (unless they're just the quick assert line).
>
> [...]
>
> Also I'm not sure from the code if you allowed both --diffs and
> --summarize to be specified together (thus displaying both the patch and
> the summary).

Travis, will you be responding to these comments from Giovanni?

Thanks,
Raman Gupta
_______________________________________________
Svnmerge mailing list
[email protected]
http://www.orcaware.com/mailman/listinfo/svnmerge

Reply via email to