Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1693327 > unrolled thread
| Started by | frowand.list@gmail.com |
|---|---|
| First post | 2017-07-21 03:40 +0200 |
| Last post | 2017-07-21 22:50 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] docs: submitting-patches - change non-ascii character to ascii frowand.list@gmail.com - 2017-07-21 03:40 +0200
Re: [PATCH] docs: submitting-patches - change non-ascii character to ascii Jonathan Corbet <corbet@lwn.net> - 2017-07-21 19:30 +0200
Re: [PATCH] docs: submitting-patches - change non-ascii character to ascii Frank Rowand <frowand.list@gmail.com> - 2017-07-21 22:50 +0200
| From | frowand.list@gmail.com |
|---|---|
| Date | 2017-07-21 03:40 +0200 |
| Subject | [PATCH] docs: submitting-patches - change non-ascii character to ascii |
| Message-ID | <u5v58-6JE-9@gated-at.bofh.it> |
From: Frank Rowand <frank.rowand@sony.com> Documentation/process/submitting-patches.rst contains a non-ascii character. Change it to the ascii equivalent. Signed-off-by: Frank Rowand <frank.rowand@sony.com> --- Documentation/process/submitting-patches.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst index 3e10719fee35..733478ade91b 100644 --- a/Documentation/process/submitting-patches.rst +++ b/Documentation/process/submitting-patches.rst @@ -413,7 +413,7 @@ e-mail discussions. -11) Sign your work — the Developer's Certificate of Origin +11) Sign your work - the Developer's Certificate of Origin ---------------------------------------------------------- To improve tracking of who did what, especially with patches that can -- Frank Rowand <frank.rowand@sony.com>
[toc] | [next] | [standalone]
| From | Jonathan Corbet <corbet@lwn.net> |
|---|---|
| Date | 2017-07-21 19:30 +0200 |
| Subject | Re: [PATCH] docs: submitting-patches - change non-ascii character to ascii |
| Message-ID | <u5JUt-7GX-15@gated-at.bofh.it> |
| In reply to | #1693327 |
On Thu, 20 Jul 2017 18:30:55 -0700 frowand.list@gmail.com wrote: > Documentation/process/submitting-patches.rst contains a non-ascii > character. Change it to the ascii equivalent. You should know better than to tell somebody like me that a hyphen and an m-dash are equivalent! :) I don't have any real objection to this change, but I am curious: is the m-dash creating a problem somewhere? We have plenty of non-ASCII characters in Documentation/ and beyond, why change this one? Or to put it another way, do you think we should have an ASCII-only policy for documentation files? Thanks, jon
[toc] | [prev] | [next] | [standalone]
| From | Frank Rowand <frowand.list@gmail.com> |
|---|---|
| Date | 2017-07-21 22:50 +0200 |
| Subject | Re: [PATCH] docs: submitting-patches - change non-ascii character to ascii |
| Message-ID | <u5N21-136-7@gated-at.bofh.it> |
| In reply to | #1693896 |
On 07/21/17 10:27, Jonathan Corbet wrote: > On Thu, 20 Jul 2017 18:30:55 -0700 > frowand.list@gmail.com wrote: > >> Documentation/process/submitting-patches.rst contains a non-ascii >> character. Change it to the ascii equivalent. > > You should know better than to tell somebody like me that a hyphen and an > m-dash are equivalent! :) OK, so they aren't totally equivalent, but close enough. :) Should I have said analog instead of equivalent? And would you prefer '--' to '-'? > I don't have any real objection to this change, but I am curious: is the > m-dash creating a problem somewhere? We have plenty of non-ASCII > characters in Documentation/ and beyond, why change this one? Or to put > it another way, do you think we should have an ASCII-only policy for > documentation files? Ascii is a lowest common denominator. I can view and manipulate the file with any common text editor and common text utilities (eg, cat, grep, etc) on pretty much any Linux system that I walk up to. I don't need to go to any effort to try to figure out what a non-ascii character is (which is exactly what prompted my patch -- I wanted to know what the character my patch modifies is). Yes, I can change my terminal emulator character encoding to UTF-8, and change my LANG to en_US.UTF-8. And now vi and cat show the correct m-dash character. But then how do I grep for m-dash in files? Google tells me I might be able to <ctrl> + <shift> + u hex_value_of_mdash to enter an mdash, but I sure don't know what the hex value of mdash is. Plus I need to be observant enough to notice that the string I am grepping for contains an m-dash instead of a dash. And why should I assume "en_US" as the prefix to my UTF-8 LANG? To answer your second question, I would _prefer_ ASCII-only except for cases where being limited to ASCII is restricting the ability to convey information (properly). I would reverse your question, and ask what is the added value of non-ascii characters __in cases similar to this one__, that justifies the negative impact? (Please don't answer what the added value is in cases that are not similar to this one. I know the answer to that question is different.) > Thanks, > > jon >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web