Nicely done.
Especially I like the part that you explicitly set charset to
UTF-8 to the primary part.
The only two and half minor issues I might have about this are:
(1) is the type text/x-patch appropriate?
(2) is it possible to cheaply come up with a safe mime-magic,
instead of a hardcoded long string and hope it does not
clash?
You can just say "Yes it is an established practice, widely
accepted and that is what you are responding to so obviously you
can grok it ;-)" to (1). About (2), you would probably need to
read the "diff-tree -p" output beforehand if we want to be
absolutely sure, so punting on the issue like this might be the
best practical approach for now, but I am asking it anyway
because people may have better ideas.
The remaining half issue is if would it make sense to sometimes
optionally use non 8-bit CTE for the patch part. I do _NOT_
want to receive CTE=QP patch myself, nor I want to encourage it
(actually I would want to actively discourage it), but I do not
mind if people find use of such a patch in a distant corner of
the galaxy where I do not have to touch such a patch.
-
: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html