unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: [Emacs-diffs] master efa6f10: .gitignore tweaks
       [not found] ` <E1YwXVp-0006ht-UU@vcs.savannah.gnu.org>
@ 2015-05-24 15:10   ` Dmitry Gutov
  2015-05-24 17:30     ` Paul Eggert
  0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Gutov @ 2015-05-24 15:10 UTC (permalink / raw)
  To: emacs-devel, Paul Eggert

On 05/24/2015 06:08 PM, Paul Eggert wrote:

>      Ignore [0-9]*.txt (commonly used name for git patches)

Where does that convention come from? I wish people didn't use .txt for 
patches.




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Emacs-diffs] master efa6f10: .gitignore tweaks
  2015-05-24 15:10   ` [Emacs-diffs] master efa6f10: .gitignore tweaks Dmitry Gutov
@ 2015-05-24 17:30     ` Paul Eggert
  2015-05-24 18:09       ` Dmitry Gutov
  0 siblings, 1 reply; 4+ messages in thread
From: Paul Eggert @ 2015-05-24 17:30 UTC (permalink / raw)
  To: Dmitry Gutov, emacs-devel; +Cc: Ivan Shmakov

[-- Attachment #1: Type: text/plain, Size: 2077 bytes --]

Dmitry Gutov wrote:
> Where does that convention come from? I wish people didn't use .txt for patches.

Using .txt for attached patches works around a Thunderbird bug noted by Ivan 
Shmakov on this list.  See:

http://bugs.gnu.org/20499#108
https://lists.gnu.org/archive/html/emacs-devel/2015-05/msg00416.html

You can see the problem in the email I'm sending to you now.  It has two 
attachments of identical files with different names, one ending in ".patch" and 
the other in ".txt".

The .patch attachment has a header like this:

Content-Type: text/x-patch;
  name="0045-Prefer-this-to-this-in-doc-strings.patch"
Content-Transfer-Encoding: 8bit
Content-Disposition: attachment;
  filename="0045-Prefer-this-to-this-in-doc-strings.patch"

and contents that start like this:

 From f743819b57ef519109c1b9d520d358d19a197086 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Tue, 19 May 2015 19:08:11 -0700
Subject: [PATCH 045/100] =?UTF-8?q?Prefer=20"this"=20to=20=E2=80=9Cthis?=
  =?UTF-8?q?=E2=80=9D=20in=20doc=20strings?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Here, Thunderbird has given up on worrying about encoding, and is just passing 
it through without specifying encoding.  The contents are another email, which 
has a proper header.  This doesn't conform well to the relevant RFCs, since it's 
a text attachment that isn't ASCII but doesn't declare its charset.


The .txt attachment has a header like this:

Content-Type: text/plain; charset=UTF-8;
  name="0045-Prefer-this-to-this-in-doc-strings.txt"
Content-Transfer-Encoding: base64
Content-Disposition: attachment;
  filename="0045-Prefer-this-to-this-in-doc-strings.txt"

and contents that start like this:

RnJvbSBmNzQzODE5YjU3ZWY1MTkxMDljMWI5ZDUyMGQzNThkMTlhMTk3MDg2IE1vbiBTZXAg
MTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBQYXVsIEVnZ2VydCA8ZWdnZXJ0QGNzLnVjbGEuZWR1

Here, the contents are a base64 representation of exactly the same file as 
before, but Thunderbird has labeled the attachment as UTF-8.  This is more 
likely to work in practice.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0045-Prefer-this-to-this-in-doc-strings.patch --]
[-- Type: text/x-patch; name="0045-Prefer-this-to-this-in-doc-strings.patch", Size: 15178 bytes --]

From f743819b57ef519109c1b9d520d358d19a197086 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Tue, 19 May 2015 19:08:11 -0700
Subject: [PATCH 045/100] =?UTF-8?q?Prefer=20"this"=20to=20=E2=80=9Cthis?=
 =?UTF-8?q?=E2=80=9D=20in=20doc=20strings?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This mostly just straightens quotes introduced in my previous patch.
Suggested by Dmitry Gutov in:
http://lists.gnu.org/archive/html/emacs-devel/2015-05/msg00565.html
* lisp/faces.el, lisp/gnus/gnus-group.el, lisp/ldefs-boot.el:
* lisp/mail/supercite.el, lisp/net/tramp.el, lisp/recentf.el:
* lisp/textmodes/artist.el, lisp/textmodes/rst.el:
* lisp/textmodes/tildify.el, lisp/vc/ediff-util.el:
* lisp/vc/log-edit.el, lisp/xt-mouse.el:
Prefer straight double quotes to curved double quotes in doc strings.
---
 lisp/faces.el             |  4 ++--
 lisp/gnus/gnus-group.el   |  2 +-
 lisp/ldefs-boot.el        |  2 +-
 lisp/mail/supercite.el    | 10 +++++-----
 lisp/net/tramp.el         |  4 ++--
 lisp/recentf.el           |  2 +-
 lisp/textmodes/artist.el  | 24 ++++++++++++------------
 lisp/textmodes/rst.el     |  4 ++--
 lisp/textmodes/tildify.el |  2 +-
 lisp/vc/ediff-util.el     |  2 +-
 lisp/vc/log-edit.el       |  2 +-
 lisp/xt-mouse.el          |  8 ++++----
 12 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/lisp/faces.el b/lisp/faces.el
index d6d3766..9c087c9 100644
--- a/lisp/faces.el
+++ b/lisp/faces.el
@@ -892,7 +892,7 @@ where COLOR is a string or `foreground-color', and STYLE is either
 foreground color.  :style may be omitted, which means to use a line.
 
 FRAME nil or not specified means change face on all frames.
-Use `set-face-attribute' to “unspecify” underlining."
+Use `set-face-attribute' to \"unspecify\" underlining."
   (interactive (read-face-and-attribute :underline))
   (set-face-attribute face frame :underline underline))
 
@@ -905,7 +905,7 @@ Use `set-face-attribute' to “unspecify” underlining."
 INVERSE-VIDEO-P non-nil means FACE displays explicitly in inverse video.
 INVERSE-VIDEO-P nil means FACE explicitly is not in inverse video.
 FRAME nil or not specified means change face on all frames.
-Use `set-face-attribute' to “unspecify” the inverse video attribute."
+Use `set-face-attribute' to \"unspecify\" the inverse video attribute."
   (interactive
    (let ((list (read-face-and-attribute :inverse-video)))
      (list (car list) (if (cadr list) t))))
diff --git a/lisp/gnus/gnus-group.el b/lisp/gnus/gnus-group.el
index 69a2482..dbeada2 100644
--- a/lisp/gnus/gnus-group.el
+++ b/lisp/gnus/gnus-group.el
@@ -3285,7 +3285,7 @@ mail messages or news articles in files that have numeric names."
 
    Rights is a string listing a (possibly empty) set of alphanumeric
    characters, each character listing a set of operations which is being
-   controlled.  Letters are reserved for “standard” rights, listed
+   controlled.  Letters are reserved for \"standard\" rights, listed
    below.  Digits are reserved for implementation or site defined rights.
 
    l - lookup (mailbox is visible to LIST/LSUB commands)
diff --git a/lisp/ldefs-boot.el b/lisp/ldefs-boot.el
index 0d1940c..a52a19e 100644
--- a/lisp/ldefs-boot.el
+++ b/lisp/ldefs-boot.el
@@ -27708,7 +27708,7 @@ Convert space before point into a hard space if the context is right.
 
 If
  * character before point is a space character,
- * character before that has “w” character syntax (i.e. it's a word
+ * character before that has \"w\" character syntax (i.e. it's a word
    constituent),
  * `tildify-space-pattern' matches when `looking-back' (no more than 10
    characters) from before the space character, and
diff --git a/lisp/mail/supercite.el b/lisp/mail/supercite.el
index f6b27c2..56158cc 100644
--- a/lisp/mail/supercite.el
+++ b/lisp/mail/supercite.el
@@ -886,10 +886,10 @@ Action can be one of: View, Modify, Add, or Delete."
 
 (defun sc-attribs-%@-addresses (from &optional delim)
   "Extract the author's email terminus from email address FROM.
-Match addresses of the style “name%[stuff].” when called with DELIM
-of \"%\" and addresses of the style “[stuff]name@[stuff]” when
+Match addresses of the style \"name%[stuff].\" when called with DELIM
+of \"%\" and addresses of the style \"[stuff]name@[stuff]\" when
 called with DELIM \"@\".  If DELIM is nil or not provided, matches
-addresses of the style “name”."
+addresses of the style \"name\"."
   (and (string-match (concat "[-[:alnum:]_.]+" delim) from 0)
        (substring from
 		  (match-beginning 0)
@@ -897,7 +897,7 @@ addresses of the style “name”."
 
 (defun sc-attribs-!-addresses (from)
   "Extract the author's email terminus from email address FROM.
-Match addresses of the style “[stuff]![stuff]...!name[stuff].”"
+Match addresses of the style \"[stuff]![stuff]...!name[stuff].\""
   (let ((eos (length from))
 	(mstart (string-match "![-[:alnum:]_.]+\\([^-![:alnum:]_.]\\|$\\)"
 			      from 0))
@@ -907,7 +907,7 @@ Match addresses of the style “[stuff]![stuff]...!name[stuff].”"
 
 (defun sc-attribs-<>-addresses (from)
   "Extract the author's email terminus from email address FROM.
-Match addresses of the style “<name[stuff]>.”"
+Match addresses of the style \"<name[stuff]>.\""
   (and (string-match "<\\(.*\\)>" from)
        (match-string 1 from)))
 
diff --git a/lisp/net/tramp.el b/lisp/net/tramp.el
index c71cf5e..f4ece90 100644
--- a/lisp/net/tramp.el
+++ b/lisp/net/tramp.el
@@ -300,8 +300,8 @@ are fit for gateways must have `tramp-default-port' at least.
 
 Notes:
 
-When using `su' or `sudo' the phrase “open connection to a remote
-host” sounds strange, but it is used nevertheless, for consistency.
+When using `su' or `sudo' the phrase \"open connection to a remote
+host\" sounds strange, but it is used nevertheless, for consistency.
 No connection is opened to a remote host, but `su' or `sudo' is
 started on the local host.  You should specify a remote host
 `localhost' or the name of the local host.  Another host name is
diff --git a/lisp/recentf.el b/lisp/recentf.el
index dd5ba86..89c1517 100644
--- a/lisp/recentf.el
+++ b/lisp/recentf.el
@@ -294,7 +294,7 @@ They are successively passed a file name to transform it."
             (function :tag "Other function")))))
 
 (defcustom recentf-show-file-shortcuts-flag t
-  "Whether to show “[N]” for the Nth item up to 10.
+  "Whether to show \"[N]\" for the Nth item up to 10.
 If non-nil, `recentf-open-files' will show labels for keys that can be
 used as shortcuts to open the Nth file."
   :group 'recentf
diff --git a/lisp/textmodes/artist.el b/lisp/textmodes/artist.el
index 7c9002b..14cf402 100644
--- a/lisp/textmodes/artist.el
+++ b/lisp/textmodes/artist.el
@@ -298,7 +298,7 @@ during the flood-fill."
 (defcustom artist-ellipse-right-char ?\)
   "Character to use at the rightmost position when drawing narrow ellipses.
 
-In this figure, it is the right parenthesis (the “)” character):
+In this figure, it is the right parenthesis (the \")\" character):
              -----
             (     )
              -----"
@@ -309,7 +309,7 @@ In this figure, it is the right parenthesis (the “)” character):
 (defcustom artist-ellipse-left-char ?\(
   "Character to use at the leftmost position when drawing narrow ellipses.
 
-In this figure, it is the left parenthesis (the “(” character):
+In this figure, it is the left parenthesis (the \"(\" character):
              -----
             (     )
              -----"
@@ -331,7 +331,7 @@ Accept this many characters cutting off a line and still treat
 it as one line.
 Example:
  If `artist-vaporize-fuzziness' is 2, then those will be recognized as
- lines from A to B (provided you start vaporizing them at the “*”):
+ lines from A to B (provided you start vaporizing them at the \"*\"):
                          /
             A----*------/-----------B
                       \\/
@@ -342,7 +342,7 @@ Example:
                       \\/ /
             A----*----/\\/----------B
                      / /\\
- (in fact, only the left part [between the A and the leftmost “/”
+ (in fact, only the left part [between the A and the leftmost \"/\"
  crossing the line] will be vaporized)."
   :group 'artist
   :type 'integer)
@@ -397,8 +397,8 @@ Example:
   ;; This is a defvar, not a defcustom, since the custom
   ;; package shows lists of characters as a lists of integers,
   ;; which is confusing
-  "Characters (“color”) to use when spraying.
-They should be ordered from the “lightest” to the “heaviest”
+  "Characters (\"color\") to use when spraying.
+They should be ordered from the \"lightest\" to the \"heaviest\"
 since spraying replaces a light character with the next heavier one.")
 
 
@@ -1275,7 +1275,7 @@ Drawing with the mouse:
 		* Cut copies, then clears the rectangle/square.
 
 		* When drawing lines or poly-lines, you can set arrows.
-		  See below under “Arrows” for more info.
+		  See below under \"Arrows\" for more info.
 
 		* The mode line shows the currently selected drawing operation.
 		  In addition, if it has an asterisk (*) at the end, you
@@ -1383,8 +1383,8 @@ Variables
  artist-vaporize-fuzziness      Tolerance when recognizing lines
  artist-spray-interval          Seconds between repeated sprayings
  artist-spray-radius            Size of the spray-area
- artist-spray-chars             The spray-“color”
- artist-spray-new-chars         Initial spray-“color”
+ artist-spray-chars             The spray-\"color\"
+ artist-spray-new-chars         Initial spray-\"color\"
 
 Hooks
 
@@ -2020,7 +2020,7 @@ The replacement is used to convert tabs and new-lines to spaces."
 (defsubst artist-replace-string (string &optional see-thru)
   "Replace contents at point with STRING.
 With optional argument SEE-THRU set to non-nil, text in the buffer
-“shines thru” blanks in the STRING."
+\"shines thru\" blanks in the STRING."
   (let ((char-list (append string nil))	; convert the string to a list
 	(overwrite-mode 'overwrite-mode-textual)
 	(fill-column 32765)		; Large :-)
@@ -2385,7 +2385,7 @@ in the coord."
 ;; Pretend we are plotting a pixel. Instead we just list it
 ;;
 (defmacro artist-put-pixel (point-list x y)
-  "In POINT-LIST, store a “pixel” at coord X,Y."
+  "In POINT-LIST, store a \"pixel\" at coord X,Y."
   `(setq ,point-list
 	 (append ,point-list (list (artist-new-coord ,x ,y)))))
 
@@ -2928,7 +2928,7 @@ This is done by calling the function specified by
 `artist-text-renderer-function', which must return a list of strings,
 to be inserted in the buffer.
 
-Text already in the buffer “shines thru” blanks in the rendered text."
+Text already in the buffer \"shines thru\" blanks in the rendered text."
   (let* ((input-text (read-string "Type text to render: "))
 	 (rendered-text (artist-funcall artist-text-renderer-function input-text)))
     (artist-text-insert-see-thru x y rendered-text)))
diff --git a/lisp/textmodes/rst.el b/lisp/textmodes/rst.el
index bc437b7..9d3f2a9 100644
--- a/lisp/textmodes/rst.el
+++ b/lisp/textmodes/rst.el
@@ -1512,8 +1512,8 @@ PFXARG (alternate behavior).
 
 This function is a bit of a swiss knife.  It is meant to adjust
 the adornments of a section title in reStructuredText.  It tries
-to deal with all the possible cases gracefully and to do “the
-right thing” in all cases.
+to deal with all the possible cases gracefully and to do \"the
+right thing\" in all cases.
 
 See the documentations of `rst-adjust-adornment-work' and
 `rst-promote-region' for full details.
diff --git a/lisp/textmodes/tildify.el b/lisp/textmodes/tildify.el
index 0eae67a..4e385a0 100644
--- a/lisp/textmodes/tildify.el
+++ b/lisp/textmodes/tildify.el
@@ -431,7 +431,7 @@ current `case-fold-search' setting."
 
 If
  * character before point is a space character,
- * character before that has “w” character syntax (i.e. it's a word
+ * character before that has \"w\" character syntax (i.e. it's a word
    constituent),
  * `tildify-space-pattern' matches when `looking-back' (no more than 10
    characters) from before the space character, and
diff --git a/lisp/vc/ediff-util.el b/lisp/vc/ediff-util.el
index 4ebffec..26bba40 100644
--- a/lisp/vc/ediff-util.el
+++ b/lisp/vc/ediff-util.el
@@ -3988,7 +3988,7 @@ and mail it to the address above.
 Please read this first:
 ----------------------
 
-Some “bugs” may actually be no bugs at all.  For instance, if you are
+Some \"bugs\" may actually be no bugs at all.  For instance, if you are
 reporting that certain difference regions are not matched as you think they
 should, this is most likely due to the way Unix diff program decides what
 constitutes a difference region.  Ediff is an Emacs interface to diff, and
diff --git a/lisp/vc/log-edit.el b/lisp/vc/log-edit.el
index 452977a..d595497 100644
--- a/lisp/vc/log-edit.el
+++ b/lisp/vc/log-edit.el
@@ -167,7 +167,7 @@ can be obtained from `log-edit-files'."
 
 (defvar log-edit-changelog-full-paragraphs t
   "If non-nil, include full ChangeLog paragraphs in the log.
-This may be set in the “local variables” section of a ChangeLog, to
+This may be set in the \"local variables\" section of a ChangeLog, to
 indicate the policy for that ChangeLog.
 
 A ChangeLog paragraph is a bunch of log text containing no blank lines;
diff --git a/lisp/xt-mouse.el b/lisp/xt-mouse.el
index 5a258a7..2bb7154 100644
--- a/lisp/xt-mouse.el
+++ b/lisp/xt-mouse.el
@@ -321,21 +321,21 @@ Enables basic mouse tracking, mouse motion events and finally
 extended tracking on terminals that support it. The following
 escape sequences are understood by modern xterms:
 
-\"\\e[?1000h\" “Basic mouse mode”: Enables reports for mouse
+\"\\e[?1000h\" \"Basic mouse mode\": Enables reports for mouse
             clicks. There is a limit to the maximum row/column
             position (<= 223), which can be reported in this
             basic mode.
 
-\"\\e[?1002h\" “Mouse motion mode”: Enables reports for mouse
+\"\\e[?1002h\" \"Mouse motion mode\": Enables reports for mouse
             motion events during dragging operations.
 
-\"\\e[?1005h\" “UTF-8 coordinate extension”: Enables an extension
+\"\\e[?1005h\" \"UTF-8 coordinate extension\": Enables an extension
             to the basic mouse mode, which uses UTF-8
             characters to overcome the 223 row/column limit. This
             extension may conflict with non UTF-8 applications or
             non UTF-8 locales.
 
-\"\\e[?1006h\" “SGR coordinate extension”: Enables a newer
+\"\\e[?1006h\" \"SGR coordinate extension\": Enables a newer
             alternative extension to the basic mouse mode, which
             overcomes the 223 row/column limit without the
             drawbacks of the UTF-8 coordinate extension.
-- 
2.1.0


[-- Attachment #3: 0045-Prefer-this-to-this-in-doc-strings.txt --]
[-- Type: text/plain, Size: 14848 bytes --]

From f743819b57ef519109c1b9d520d358d19a197086 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Tue, 19 May 2015 19:08:11 -0700
Subject: [PATCH 045/100] =?UTF-8?q?Prefer=20"this"=20to=20=E2=80=9Cthis?=
 =?UTF-8?q?=E2=80=9D=20in=20doc=20strings?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This mostly just straightens quotes introduced in my previous patch.
Suggested by Dmitry Gutov in:
http://lists.gnu.org/archive/html/emacs-devel/2015-05/msg00565.html
* lisp/faces.el, lisp/gnus/gnus-group.el, lisp/ldefs-boot.el:
* lisp/mail/supercite.el, lisp/net/tramp.el, lisp/recentf.el:
* lisp/textmodes/artist.el, lisp/textmodes/rst.el:
* lisp/textmodes/tildify.el, lisp/vc/ediff-util.el:
* lisp/vc/log-edit.el, lisp/xt-mouse.el:
Prefer straight double quotes to curved double quotes in doc strings.
---
 lisp/faces.el             |  4 ++--
 lisp/gnus/gnus-group.el   |  2 +-
 lisp/ldefs-boot.el        |  2 +-
 lisp/mail/supercite.el    | 10 +++++-----
 lisp/net/tramp.el         |  4 ++--
 lisp/recentf.el           |  2 +-
 lisp/textmodes/artist.el  | 24 ++++++++++++------------
 lisp/textmodes/rst.el     |  4 ++--
 lisp/textmodes/tildify.el |  2 +-
 lisp/vc/ediff-util.el     |  2 +-
 lisp/vc/log-edit.el       |  2 +-
 lisp/xt-mouse.el          |  8 ++++----
 12 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/lisp/faces.el b/lisp/faces.el
index d6d3766..9c087c9 100644
--- a/lisp/faces.el
+++ b/lisp/faces.el
@@ -892,7 +892,7 @@ where COLOR is a string or `foreground-color', and STYLE is either
 foreground color.  :style may be omitted, which means to use a line.
 
 FRAME nil or not specified means change face on all frames.
-Use `set-face-attribute' to “unspecify” underlining."
+Use `set-face-attribute' to \"unspecify\" underlining."
   (interactive (read-face-and-attribute :underline))
   (set-face-attribute face frame :underline underline))
 
@@ -905,7 +905,7 @@ Use `set-face-attribute' to “unspecify” underlining."
 INVERSE-VIDEO-P non-nil means FACE displays explicitly in inverse video.
 INVERSE-VIDEO-P nil means FACE explicitly is not in inverse video.
 FRAME nil or not specified means change face on all frames.
-Use `set-face-attribute' to “unspecify” the inverse video attribute."
+Use `set-face-attribute' to \"unspecify\" the inverse video attribute."
   (interactive
    (let ((list (read-face-and-attribute :inverse-video)))
      (list (car list) (if (cadr list) t))))
diff --git a/lisp/gnus/gnus-group.el b/lisp/gnus/gnus-group.el
index 69a2482..dbeada2 100644
--- a/lisp/gnus/gnus-group.el
+++ b/lisp/gnus/gnus-group.el
@@ -3285,7 +3285,7 @@ mail messages or news articles in files that have numeric names."
 
    Rights is a string listing a (possibly empty) set of alphanumeric
    characters, each character listing a set of operations which is being
-   controlled.  Letters are reserved for “standard” rights, listed
+   controlled.  Letters are reserved for \"standard\" rights, listed
    below.  Digits are reserved for implementation or site defined rights.
 
    l - lookup (mailbox is visible to LIST/LSUB commands)
diff --git a/lisp/ldefs-boot.el b/lisp/ldefs-boot.el
index 0d1940c..a52a19e 100644
--- a/lisp/ldefs-boot.el
+++ b/lisp/ldefs-boot.el
@@ -27708,7 +27708,7 @@ Convert space before point into a hard space if the context is right.
 
 If
  * character before point is a space character,
- * character before that has “w” character syntax (i.e. it's a word
+ * character before that has \"w\" character syntax (i.e. it's a word
    constituent),
  * `tildify-space-pattern' matches when `looking-back' (no more than 10
    characters) from before the space character, and
diff --git a/lisp/mail/supercite.el b/lisp/mail/supercite.el
index f6b27c2..56158cc 100644
--- a/lisp/mail/supercite.el
+++ b/lisp/mail/supercite.el
@@ -886,10 +886,10 @@ Action can be one of: View, Modify, Add, or Delete."
 
 (defun sc-attribs-%@-addresses (from &optional delim)
   "Extract the author's email terminus from email address FROM.
-Match addresses of the style “name%[stuff].” when called with DELIM
-of \"%\" and addresses of the style “[stuff]name@[stuff]” when
+Match addresses of the style \"name%[stuff].\" when called with DELIM
+of \"%\" and addresses of the style \"[stuff]name@[stuff]\" when
 called with DELIM \"@\".  If DELIM is nil or not provided, matches
-addresses of the style “name”."
+addresses of the style \"name\"."
   (and (string-match (concat "[-[:alnum:]_.]+" delim) from 0)
        (substring from
 		  (match-beginning 0)
@@ -897,7 +897,7 @@ addresses of the style “name”."
 
 (defun sc-attribs-!-addresses (from)
   "Extract the author's email terminus from email address FROM.
-Match addresses of the style “[stuff]![stuff]...!name[stuff].”"
+Match addresses of the style \"[stuff]![stuff]...!name[stuff].\""
   (let ((eos (length from))
 	(mstart (string-match "![-[:alnum:]_.]+\\([^-![:alnum:]_.]\\|$\\)"
 			      from 0))
@@ -907,7 +907,7 @@ Match addresses of the style “[stuff]![stuff]...!name[stuff].”"
 
 (defun sc-attribs-<>-addresses (from)
   "Extract the author's email terminus from email address FROM.
-Match addresses of the style “<name[stuff]>.”"
+Match addresses of the style \"<name[stuff]>.\""
   (and (string-match "<\\(.*\\)>" from)
        (match-string 1 from)))
 
diff --git a/lisp/net/tramp.el b/lisp/net/tramp.el
index c71cf5e..f4ece90 100644
--- a/lisp/net/tramp.el
+++ b/lisp/net/tramp.el
@@ -300,8 +300,8 @@ are fit for gateways must have `tramp-default-port' at least.
 
 Notes:
 
-When using `su' or `sudo' the phrase “open connection to a remote
-host” sounds strange, but it is used nevertheless, for consistency.
+When using `su' or `sudo' the phrase \"open connection to a remote
+host\" sounds strange, but it is used nevertheless, for consistency.
 No connection is opened to a remote host, but `su' or `sudo' is
 started on the local host.  You should specify a remote host
 `localhost' or the name of the local host.  Another host name is
diff --git a/lisp/recentf.el b/lisp/recentf.el
index dd5ba86..89c1517 100644
--- a/lisp/recentf.el
+++ b/lisp/recentf.el
@@ -294,7 +294,7 @@ They are successively passed a file name to transform it."
             (function :tag "Other function")))))
 
 (defcustom recentf-show-file-shortcuts-flag t
-  "Whether to show “[N]” for the Nth item up to 10.
+  "Whether to show \"[N]\" for the Nth item up to 10.
 If non-nil, `recentf-open-files' will show labels for keys that can be
 used as shortcuts to open the Nth file."
   :group 'recentf
diff --git a/lisp/textmodes/artist.el b/lisp/textmodes/artist.el
index 7c9002b..14cf402 100644
--- a/lisp/textmodes/artist.el
+++ b/lisp/textmodes/artist.el
@@ -298,7 +298,7 @@ during the flood-fill."
 (defcustom artist-ellipse-right-char ?\)
   "Character to use at the rightmost position when drawing narrow ellipses.
 
-In this figure, it is the right parenthesis (the “)” character):
+In this figure, it is the right parenthesis (the \")\" character):
              -----
             (     )
              -----"
@@ -309,7 +309,7 @@ In this figure, it is the right parenthesis (the “)” character):
 (defcustom artist-ellipse-left-char ?\(
   "Character to use at the leftmost position when drawing narrow ellipses.
 
-In this figure, it is the left parenthesis (the “(” character):
+In this figure, it is the left parenthesis (the \"(\" character):
              -----
             (     )
              -----"
@@ -331,7 +331,7 @@ Accept this many characters cutting off a line and still treat
 it as one line.
 Example:
  If `artist-vaporize-fuzziness' is 2, then those will be recognized as
- lines from A to B (provided you start vaporizing them at the “*”):
+ lines from A to B (provided you start vaporizing them at the \"*\"):
                          /
             A----*------/-----------B
                       \\/
@@ -342,7 +342,7 @@ Example:
                       \\/ /
             A----*----/\\/----------B
                      / /\\
- (in fact, only the left part [between the A and the leftmost “/”
+ (in fact, only the left part [between the A and the leftmost \"/\"
  crossing the line] will be vaporized)."
   :group 'artist
   :type 'integer)
@@ -397,8 +397,8 @@ Example:
   ;; This is a defvar, not a defcustom, since the custom
   ;; package shows lists of characters as a lists of integers,
   ;; which is confusing
-  "Characters (“color”) to use when spraying.
-They should be ordered from the “lightest” to the “heaviest”
+  "Characters (\"color\") to use when spraying.
+They should be ordered from the \"lightest\" to the \"heaviest\"
 since spraying replaces a light character with the next heavier one.")
 
 
@@ -1275,7 +1275,7 @@ Drawing with the mouse:
 		* Cut copies, then clears the rectangle/square.
 
 		* When drawing lines or poly-lines, you can set arrows.
-		  See below under “Arrows” for more info.
+		  See below under \"Arrows\" for more info.
 
 		* The mode line shows the currently selected drawing operation.
 		  In addition, if it has an asterisk (*) at the end, you
@@ -1383,8 +1383,8 @@ Variables
  artist-vaporize-fuzziness      Tolerance when recognizing lines
  artist-spray-interval          Seconds between repeated sprayings
  artist-spray-radius            Size of the spray-area
- artist-spray-chars             The spray-“color”
- artist-spray-new-chars         Initial spray-“color”
+ artist-spray-chars             The spray-\"color\"
+ artist-spray-new-chars         Initial spray-\"color\"
 
 Hooks
 
@@ -2020,7 +2020,7 @@ The replacement is used to convert tabs and new-lines to spaces."
 (defsubst artist-replace-string (string &optional see-thru)
   "Replace contents at point with STRING.
 With optional argument SEE-THRU set to non-nil, text in the buffer
-“shines thru” blanks in the STRING."
+\"shines thru\" blanks in the STRING."
   (let ((char-list (append string nil))	; convert the string to a list
 	(overwrite-mode 'overwrite-mode-textual)
 	(fill-column 32765)		; Large :-)
@@ -2385,7 +2385,7 @@ in the coord."
 ;; Pretend we are plotting a pixel. Instead we just list it
 ;;
 (defmacro artist-put-pixel (point-list x y)
-  "In POINT-LIST, store a “pixel” at coord X,Y."
+  "In POINT-LIST, store a \"pixel\" at coord X,Y."
   `(setq ,point-list
 	 (append ,point-list (list (artist-new-coord ,x ,y)))))
 
@@ -2928,7 +2928,7 @@ This is done by calling the function specified by
 `artist-text-renderer-function', which must return a list of strings,
 to be inserted in the buffer.
 
-Text already in the buffer “shines thru” blanks in the rendered text."
+Text already in the buffer \"shines thru\" blanks in the rendered text."
   (let* ((input-text (read-string "Type text to render: "))
 	 (rendered-text (artist-funcall artist-text-renderer-function input-text)))
     (artist-text-insert-see-thru x y rendered-text)))
diff --git a/lisp/textmodes/rst.el b/lisp/textmodes/rst.el
index bc437b7..9d3f2a9 100644
--- a/lisp/textmodes/rst.el
+++ b/lisp/textmodes/rst.el
@@ -1512,8 +1512,8 @@ PFXARG (alternate behavior).
 
 This function is a bit of a swiss knife.  It is meant to adjust
 the adornments of a section title in reStructuredText.  It tries
-to deal with all the possible cases gracefully and to do “the
-right thing” in all cases.
+to deal with all the possible cases gracefully and to do \"the
+right thing\" in all cases.
 
 See the documentations of `rst-adjust-adornment-work' and
 `rst-promote-region' for full details.
diff --git a/lisp/textmodes/tildify.el b/lisp/textmodes/tildify.el
index 0eae67a..4e385a0 100644
--- a/lisp/textmodes/tildify.el
+++ b/lisp/textmodes/tildify.el
@@ -431,7 +431,7 @@ current `case-fold-search' setting."
 
 If
  * character before point is a space character,
- * character before that has “w” character syntax (i.e. it's a word
+ * character before that has \"w\" character syntax (i.e. it's a word
    constituent),
  * `tildify-space-pattern' matches when `looking-back' (no more than 10
    characters) from before the space character, and
diff --git a/lisp/vc/ediff-util.el b/lisp/vc/ediff-util.el
index 4ebffec..26bba40 100644
--- a/lisp/vc/ediff-util.el
+++ b/lisp/vc/ediff-util.el
@@ -3988,7 +3988,7 @@ and mail it to the address above.
 Please read this first:
 ----------------------
 
-Some “bugs” may actually be no bugs at all.  For instance, if you are
+Some \"bugs\" may actually be no bugs at all.  For instance, if you are
 reporting that certain difference regions are not matched as you think they
 should, this is most likely due to the way Unix diff program decides what
 constitutes a difference region.  Ediff is an Emacs interface to diff, and
diff --git a/lisp/vc/log-edit.el b/lisp/vc/log-edit.el
index 452977a..d595497 100644
--- a/lisp/vc/log-edit.el
+++ b/lisp/vc/log-edit.el
@@ -167,7 +167,7 @@ can be obtained from `log-edit-files'."
 
 (defvar log-edit-changelog-full-paragraphs t
   "If non-nil, include full ChangeLog paragraphs in the log.
-This may be set in the “local variables” section of a ChangeLog, to
+This may be set in the \"local variables\" section of a ChangeLog, to
 indicate the policy for that ChangeLog.
 
 A ChangeLog paragraph is a bunch of log text containing no blank lines;
diff --git a/lisp/xt-mouse.el b/lisp/xt-mouse.el
index 5a258a7..2bb7154 100644
--- a/lisp/xt-mouse.el
+++ b/lisp/xt-mouse.el
@@ -321,21 +321,21 @@ Enables basic mouse tracking, mouse motion events and finally
 extended tracking on terminals that support it. The following
 escape sequences are understood by modern xterms:
 
-\"\\e[?1000h\" “Basic mouse mode”: Enables reports for mouse
+\"\\e[?1000h\" \"Basic mouse mode\": Enables reports for mouse
             clicks. There is a limit to the maximum row/column
             position (<= 223), which can be reported in this
             basic mode.
 
-\"\\e[?1002h\" “Mouse motion mode”: Enables reports for mouse
+\"\\e[?1002h\" \"Mouse motion mode\": Enables reports for mouse
             motion events during dragging operations.
 
-\"\\e[?1005h\" “UTF-8 coordinate extension”: Enables an extension
+\"\\e[?1005h\" \"UTF-8 coordinate extension\": Enables an extension
             to the basic mouse mode, which uses UTF-8
             characters to overcome the 223 row/column limit. This
             extension may conflict with non UTF-8 applications or
             non UTF-8 locales.
 
-\"\\e[?1006h\" “SGR coordinate extension”: Enables a newer
+\"\\e[?1006h\" \"SGR coordinate extension\": Enables a newer
             alternative extension to the basic mouse mode, which
             overcomes the 223 row/column limit without the
             drawbacks of the UTF-8 coordinate extension.
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Emacs-diffs] master efa6f10: .gitignore tweaks
  2015-05-24 17:30     ` Paul Eggert
@ 2015-05-24 18:09       ` Dmitry Gutov
  2015-05-24 18:20         ` Paul Eggert
  0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Gutov @ 2015-05-24 18:09 UTC (permalink / raw)
  To: Paul Eggert, emacs-devel; +Cc: Ivan Shmakov

On 05/24/2015 08:30 PM, Paul Eggert wrote:

 > Using .txt for attached patches works around a Thunderbird bug noted
 > by Ivan Shmakov on this list.

Interesting. I've missed that email entirely.

> Here, the contents are a base64 representation of exactly the same file
> as before, but Thunderbird has labeled the attachment as UTF-8.  This is
> more likely to work in practice.

Thanks, I see it, and it's the same in email sent from Thunderbird 38 
(beta).

The intricacies of how it should work are above me, but isn't this bug 
report-worthy?

And sending patch with a "wrong" extension means that after it's saved, 
the other party will have more trouble viewing it correctly. gedit does 
no appropriate syntax highlighting for .txt, and Emacs opens them in 
text-mode by default (which means no syntax highlighting, or chunk 
navigation commands). This cure might be worse than the disease.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Emacs-diffs] master efa6f10: .gitignore tweaks
  2015-05-24 18:09       ` Dmitry Gutov
@ 2015-05-24 18:20         ` Paul Eggert
  0 siblings, 0 replies; 4+ messages in thread
From: Paul Eggert @ 2015-05-24 18:20 UTC (permalink / raw)
  To: Dmitry Gutov, emacs-devel; +Cc: Ivan Shmakov

Dmitry Gutov wrote:
> isn't this bug report-worthy?

Yes, I've filed a Thunderbird bug report here:

https://bugzilla.mozilla.org/show_bug.cgi?id=1167982



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-05-24 18:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20150524150825.25739.79722@vcs.savannah.gnu.org>
     [not found] ` <E1YwXVp-0006ht-UU@vcs.savannah.gnu.org>
2015-05-24 15:10   ` [Emacs-diffs] master efa6f10: .gitignore tweaks Dmitry Gutov
2015-05-24 17:30     ` Paul Eggert
2015-05-24 18:09       ` Dmitry Gutov
2015-05-24 18:20         ` Paul Eggert

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).