Re: [Qemu-devel] [PATCH v3 0/7] cpu: add device_add foo-x86_64-cpu support
On Thu, Feb 05, 2015 at 12:17:15PM -0800, Junio C Hamano wrote: Would length() 1 be enough[1]? Or are people really typing yes and not just y? I cannot imagine a charset name that is smaller than two characters. It may be that there are none smaller than 4, and we could cut it off there. Googling around for some lists of common charsets, it seems like that might be plausible (but not any larger; big5 is 4 characters, and people may spell utf8 without the hyphen). -Peff [1] Of course, to match the existing regex code, we may want to spell this as /../ or //. Perhaps. Just in case there were shorter ones, something like this with confirm_only to allow them to say Yes, I do mean 'xx'? git-send-email.perl | 1 + 1 file changed, 1 insertion(+) diff --git a/git-send-email.perl b/git-send-email.perl index 3092ab3..848f176 100755 --- a/git-send-email.perl +++ b/git-send-email.perl @@ -752,6 +752,7 @@ sub file_declares_8bit_cte { print $f\n; } $auto_8bit_encoding = ask(Which 8bit encoding should I declare [UTF-8]? , + valid_re = qr/.{4}/, confirm_only = 1, default = UTF-8); } Yes, I think leaving an escape hatch is a good idea, just in case. -Peff -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Qemu-devel] [PATCH v3 0/7] cpu: add device_add foo-x86_64-cpu support
[adding git list to cc] On 02/05/2015 04:49 AM, Stefan Hajnoczi wrote: On Wed, Jan 14, 2015 at 03:27:23PM +0800, Zhu Guihua wrote: This series is based on the previous patchset from Chen Fan: https://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg02360.html This email has an invalid charset: Content-Type: text/plain; charset=y I guess you entered y when asked how the message was encoded. Please don't do that, it means we can only guess at the charset. In the past, people made a similar problem when 'git send-email' was asking if a message was in-reply-to something else (the number of messages incorrectly threaded to a message-id of 'y' or 'n' was evidence of the poor quality of the question). git.git commit 51bbccfd1b4a corrected that problem. Sounds like charset encoding is another case where the interactive parser should be taught to balk at nonsense encoding answers? -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH v3 0/7] cpu: add device_add foo-x86_64-cpu support
Eric Blake ebl...@redhat.com writes: On 02/05/2015 04:49 AM, Stefan Hajnoczi wrote: On Wed, Jan 14, 2015 at 03:27:23PM +0800, Zhu Guihua wrote: This series is based on the previous patchset from Chen Fan: https://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg02360.html This email has an invalid charset: Content-Type: text/plain; charset=y I guess you entered y when asked how the message was encoded. Please don't do that, it means we can only guess at the charset. In the past, people made a similar problem when 'git send-email' was asking if a message was in-reply-to something else (the number of messages incorrectly threaded to a message-id of 'y' or 'n' was evidence of the poor quality of the question). git.git commit 51bbccfd1b4a corrected that problem. Sounds like charset encoding is another case where the interactive parser should be taught to balk at nonsense encoding answers? I think I answered this in $gmane/263354; care to come up with a plausible valid_re? It is inpractical to attempt to cover all valid charset names, so whatever you do I'd imagine you would want to pass the confirm_only parameter set to true. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Qemu-devel] [PATCH v3 0/7] cpu: add device_add foo-x86_64-cpu support
On Thu, Feb 05, 2015 at 11:29:07AM -0800, Junio C Hamano wrote: Eric Blake ebl...@redhat.com writes: On 02/05/2015 04:49 AM, Stefan Hajnoczi wrote: On Wed, Jan 14, 2015 at 03:27:23PM +0800, Zhu Guihua wrote: This series is based on the previous patchset from Chen Fan: https://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg02360.html This email has an invalid charset: Content-Type: text/plain; charset=y I guess you entered y when asked how the message was encoded. Please don't do that, it means we can only guess at the charset. In the past, people made a similar problem when 'git send-email' was asking if a message was in-reply-to something else (the number of messages incorrectly threaded to a message-id of 'y' or 'n' was evidence of the poor quality of the question). git.git commit 51bbccfd1b4a corrected that problem. Sounds like charset encoding is another case where the interactive parser should be taught to balk at nonsense encoding answers? I think I answered this in $gmane/263354; care to come up with a plausible valid_re? It is inpractical to attempt to cover all valid charset names, so whatever you do I'd imagine you would want to pass the confirm_only parameter set to true. Would length() 1 be enough[1]? Or are people really typing yes and not just y? I cannot imagine a charset name that is smaller than two characters. It may be that there are none smaller than 4, and we could cut it off there. Googling around for some lists of common charsets, it seems like that might be plausible (but not any larger; big5 is 4 characters, and people may spell utf8 without the hyphen). -Peff [1] Of course, to match the existing regex code, we may want to spell this as /../ or //. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Qemu-devel] [PATCH v3 0/7] cpu: add device_add foo-x86_64-cpu support
Jeff King p...@peff.net writes: On Thu, Feb 05, 2015 at 11:29:07AM -0800, Junio C Hamano wrote: Eric Blake ebl...@redhat.com writes: On 02/05/2015 04:49 AM, Stefan Hajnoczi wrote: On Wed, Jan 14, 2015 at 03:27:23PM +0800, Zhu Guihua wrote: This series is based on the previous patchset from Chen Fan: https://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg02360.html This email has an invalid charset: Content-Type: text/plain; charset=y I guess you entered y when asked how the message was encoded. Please don't do that, it means we can only guess at the charset. In the past, people made a similar problem when 'git send-email' was asking if a message was in-reply-to something else (the number of messages incorrectly threaded to a message-id of 'y' or 'n' was evidence of the poor quality of the question). git.git commit 51bbccfd1b4a corrected that problem. Sounds like charset encoding is another case where the interactive parser should be taught to balk at nonsense encoding answers? I think I answered this in $gmane/263354; care to come up with a plausible valid_re? It is inpractical to attempt to cover all valid charset names, so whatever you do I'd imagine you would want to pass the confirm_only parameter set to true. Would length() 1 be enough[1]? Or are people really typing yes and not just y? I cannot imagine a charset name that is smaller than two characters. It may be that there are none smaller than 4, and we could cut it off there. Googling around for some lists of common charsets, it seems like that might be plausible (but not any larger; big5 is 4 characters, and people may spell utf8 without the hyphen). -Peff [1] Of course, to match the existing regex code, we may want to spell this as /../ or //. Perhaps. Just in case there were shorter ones, something like this with confirm_only to allow them to say Yes, I do mean 'xx'? git-send-email.perl | 1 + 1 file changed, 1 insertion(+) diff --git a/git-send-email.perl b/git-send-email.perl index 3092ab3..848f176 100755 --- a/git-send-email.perl +++ b/git-send-email.perl @@ -752,6 +752,7 @@ sub file_declares_8bit_cte { print $f\n; } $auto_8bit_encoding = ask(Which 8bit encoding should I declare [UTF-8]? , + valid_re = qr/.{4}/, confirm_only = 1, default = UTF-8); } -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html