Re: [PATCH] vfs: update atimes over one day in the past or future

From: Jan Engelhardt
Date: Tue Dec 25 2012 - 23:33:21 EST



On Tuesday 2012-12-18 22:14, Dave Chinner wrote:
>
>> >> CC: stable@xxxxxxxxxxxxxxx
>> >> ---
>> >> fs/inode.c | 7 ++++---
>> >> 1 ??????????????????????????? 4 ???(+)????????? 3 ???(-)
>> >
>> > There's something wrong with the character encoding you are using...
>>
>> Chinese locale, but probably doesn't matter since text below "---" isn't in the commit anyway?
>
>Yup, but it doesn't inspire confidence that the patch is going to be
>clean when multiple encodings appear in the one message...

Multiple encodings are not much of a problem. Lack of specifying them,
as happened with that attachment, is.

That's another reason why inline patches work better, because the
main message part has the encoding set most of the time :)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/