# git-mailinfo: cut lines at ^M 3 messages from 2006-01-16 to 2006-01-16. Participants: Linus Torvalds, Junio C Hamano. Thread: https://gitlist.dev/t/3086 ## Linus Torvalds, 2006-01-16 19:13 Subject: git-mailinfo: cut lines at ^M Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.64.0601161104070.13339%40g5.osdl.org ``` This changes "eatspace()" to cut lines at the first ^M (\r, CR) character. Normally it should make no difference at all, since even if you have a non-UNIX CR/LF end-of-line, it would have removed the ^M at the end of the line as whitespace. The main reason for this patch is that I seem to have something buggy in my mail path (possibly fetchmail) which very occasionally causes the mail headers to have an appended "^M)" line. This silly thing works around it, and shouldn't hurt anything else. The alternative would be to simply disallow control characters in the commit message (so that I would not commit these things by mistake). Signed-off-by: Linus Torvalds --- Junio, feel free to ignore this. It shouldn't hurt, but admittedly it's a stupid hack for a problem that has nothing to do with git. The "disallow control characters" fix is probably better, if you prefer that I can do it that way instead. Linus diff --git a/mailinfo.c b/mailinfo.c index 0265a29..e1c95a9 100644 --- a/mailinfo.c +++ b/mailinfo.c @@ -226,9 +226,19 @@ static int is_multipart_boundary(const c static int eatspace(char *line) { - int len = strlen(line); - while (len > 0 && isspace(line[len-1])) - line[--len] = 0; + int len, idx; + + len = idx = 0; + for (;;) { + unsigned char c = line[idx++]; + if (isspace(c)) { + if (c == '\n' || c == '\r') + break; + continue; + } + len = idx; + } + line[len] = 0; return len; } ``` ## Junio C Hamano, 2006-01-16 23:38 Subject: Re: git-mailinfo: cut lines at ^M Message-ID: <7v1wz7n3oz.fsf@assigned-by-dhcp.cox.net> URL: https://gitlist.dev/e/7v1wz7n3oz.fsf%40assigned-by-dhcp.cox.net In-Reply-To: ``` Linus Torvalds writes: > Junio, feel free to ignore this. It shouldn't hurt, but > admittedly it's a stupid hack for a problem that has nothing > to do with git... This is doing "instead of removing trailing spaces, terminate a line at the first CR or LF and discard the rest". So your "^M)" at the end, especially the close paren, is dropped. I've briefly wondered if a better alternative is to split lines at "\r\n", "\n", or "\r", which would make the next line begin with ")" in your example. Normally the first two would have already been done by fgets(), so the alternative approach would involve wrapping the use of fgets() with something of our own. I do not think "keeping the rest of the line after '\r' in the middle" is worth that trouble, so I am personally fine with your patch. ``` ## Linus Torvalds, 2006-01-16 23:49 Subject: Re: git-mailinfo: cut lines at ^M Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.64.0601161548560.13339%40g5.osdl.org In-Reply-To: <7v1wz7n3oz.fsf@assigned-by-dhcp.cox.net> ``` On Mon, 16 Jan 2006, Junio C Hamano wrote: > > I've briefly wondered if a better alternative is to split lines > at "\r\n", "\n", or "\r", which would make the next line begin > with ")" in your example. In which case the patch does nothing good for me. I want to either drop the bogus line, or have applymbox refuse to apply such an email.. Linus ```