Mattias Brändström wrote: >> I have written a small patch that makes avail and integrated accept a >> URL as their PATH argument. No test case yet. I just wanted to show >> the >> patch early so that I might get some comments on the small changes it >> introduces. I took Giovanni's advice and made the ability to accept >> an >> URL as PATH argument a flag in the command table.
Thanks. Please put a comment on each line containing True/False in the options, so that we know what the boolean flag means. It's not immediately obvious which stands for "accepts URL". Another solution is to change it to a whitespace-separated string, so that it is self-descriptive and allows other variations in the future (eg: "WC", "WC URL"", "URL"). -- Giovanni Bajo _______________________________________________ Svnmerge mailing list [email protected] http://www.orcaware.com/mailman/listinfo/svnmerge
