Re: [PATCH-REVIEW] Relative URL support for info command
[Troy Curtis Jr]
> Any and all comments are appreciated!
I don't have any comments on the patch itself, except to wonder why you
duplicated the strncmp("^/", path, 2) call. But I note that your mail
client labeled the patch as MIME type "Application/octet-stream". It
is much easier to review and comment on if it is labeled as
"text/plain" or, indeed, anything beginning with "text/". Many mail
clients will then let you view the patch inline and quote it in
If you don't know of any better way to control the MIME type of your
attachments, you can usually convince your mail client to use
"text/plain" by naming the file something that ends in ".txt".
Peter Samuelson | org-tld!p12n!peter | http://p12n.org/
Received on Sat Oct 27 22:06:03 2007
This is an archived mail posted to the Subversion Dev