unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
@ 2020-02-20  4:54 Roland Winkler
       [not found] ` <handler.39686.B.158217451717286.ack@debbugs.gnu.org>
  2020-02-21 16:05 ` Mattias Engdegård
  0 siblings, 2 replies; 15+ messages in thread
From: Roland Winkler @ 2020-02-20  4:54 UTC (permalink / raw)
  To: 39686; +Cc: gojjoe2

For the records:

The bug report below was submitted to auctex (bug#39479).
I am reposting it here.

==================================================================
Hi,

The bibtex-generate-autokey function uses
'bibtex-autokey-name-change-strings' to substitute special or accented
characters or ligatures with ascii characters.

I noticed that it doesn't lead to the intended behaviour for '\oe' and
'\OE', which get converted to 'oee' rather than 'oe'. On the other
hand, '\o', '\"o', and their capitalized counterparts are correctly
converted to 'oe' (and also '\ae' to 'ae').

This quirk seems to be fixed if '\o' and '\oe' are swapped in
bibtex-autokey-name-change-strings. Then all variants are correctly
converted.

So I propose to change the current bibtex-autokey-name-change-strings into

'(("\\\\aa" . "a")
  ("\\\\AA" . "A")
  ("\\\"a\\|\\\\\\\"a\\|\\\\ae" . "ae")
  ("\\\"A\\|\\\\\\\"A\\|\\\\AE" . "Ae")
  ("\\\\i" . "i")
  ("\\\\j" . "j")
  ("\\\\l" . "l")
  ("\\\\L" . "L")
  ("\\\"o\\|\\\\\\\"o\\|\\\\oe\\|\\\\o" . "oe")
  ("\\\"O\\|\\\\\\\"O\\|\\\\OE\\|\\\\O" . "Oe")
  ("\\\"s\\|\\\\\\\"s\\|\\\\3" . "ss")
  ("\\\"u\\|\\\\\\\"u" . "ue")
  ("\\\"U\\|\\\\\\\"U" . "Ue")
  ("\\\\`\\|\\\\'\\|\\\\\\^\\|\\\\~\\|\\\\=\\|\\\\\\.\\|\\\\u\\|\\\\v\\|\\\\H\\|\\\\t\\|\\\\c\\|\\\\d\\|\\\\b"
. "")
  ("[`'\"{}#]" . "")
  ("\\\\-" . "")
  ("\\\\?[ 	\n]+\\|~" . " "))

Cheers!





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
       [not found] ` <handler.39686.B.158217451717286.ack@debbugs.gnu.org>
@ 2020-02-20  5:04   ` Roland Winkler
  2020-02-21  9:22     ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2020-02-21 14:13     ` Roland Winkler
  0 siblings, 2 replies; 15+ messages in thread
From: Roland Winkler @ 2020-02-20  5:04 UTC (permalink / raw)
  To: 39686; +Cc: gojjoe2

> I noticed that it doesn't lead to the intended behaviour for '\oe'
> and '\OE', which get converted to 'oee' rather than 'oe'. On the
> other hand, '\o', '\"o', and their capitalized counterparts are
> correctly converted to 'oe' (and also '\ae' to 'ae').
>
> This quirk seems to be fixed if '\o' and '\oe' are swapped in
> bibtex-autokey-name-change-strings. Then all variants are
> correctly converted.

I suggest that the value of bibtex-autokey-transcriptions should be
calculated by regexp-opt.  That makes the code more readable and it
fixes this problem for free.

Can you confirm that the following works as it should?


(defvar bibtex-autokey-transcriptions
  (nconc
   (mapcar (lambda (a) (cons (regexp-opt (car a)) (cdr a)))
           '(;; language specific characters
             (("\\aa") . "a")                      ; \aa           -> a
             (("\\AA") . "A")                      ; \AA           -> A
             (("\"a" "\\\"a" "\\ae") . "ae")       ; "a,\"a,\ae    -> ae
             (("\"A" "\\\"A" "\\AE") . "Ae")       ; "A,\"A,\AE    -> Ae
             (("\\i") . "i")                       ; \i            -> i
             (("\\j") . "j")                       ; \j            -> j
             (("\\l") . "l")                       ; \l            -> l
             (("\\L") . "L")                       ; \L            -> L
             (("\"o" "\\\"o" "\\o" "\\oe") . "oe") ; "o,\"o,\o,\oe -> oe
             (("\"O" "\\\"O" "\\O" "\\OE") . "Oe") ; "O,\"O,\O,\OE -> Oe
             (("\"s" "\\\"s" "\\3") . "ss")        ; "s,\"s,\3     -> ss
             (("\"u" "\\\"u") . "ue")              ; "u,\"u        -> ue
             (("\"U" "\\\"U") . "Ue")              ; "U,\"U        -> Ue
             ;; hyphen, accents
             (("\\-" "\\`" "\\'" "\\^" "\\~" "\\=" "\\." "\\u" "\\v"
               "\\H" "\\t" "\\c" "\\d" "\\b") . "")
             ;; space
             (("~") . " ")))
   ;; more spaces
   '(("[\s\t\n]*\\(?:\\\\\\)?[\s\t\n]+" . " ")
     ;; braces, quotes, concatenation.
     ("[`'\"{}#]" . "")))
  "Alist of (OLD-REGEXP . NEW-STRING) pairs.
Used by the default values of `bibtex-autokey-name-change-strings' and
`bibtex-autokey-titleword-change-strings'.  Defaults to translating some
language specific characters to their ASCII transcriptions, and
removing any character accents.")





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-20  5:04   ` Roland Winkler
@ 2020-02-21  9:22     ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2020-02-21 14:06       ` Roland Winkler
  2020-02-21 14:13     ` Roland Winkler
  1 sibling, 1 reply; 15+ messages in thread
From: gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2020-02-21  9:22 UTC (permalink / raw)
  To: 39686; +Cc: Roland Winkler

Hi Roland, thank you for looking into this.

> I suggest that the value of bibtex-autokey-transcriptions should be
> calculated by regexp-opt.  That makes the code more readable and it
> fixes this problem for free.
> 
> Can you confirm that the following works as it should?

Yes it does, at least for the specific problem I found with "\oe". Thank you very much!

Can you tell me where to insert this redefinition in my init file, while I wait for the next Emacs version? I tried adding it to my own bibtex-hook and to "eval-after-load 'bibtex ...", but it doesn't seem to have any effect in either. At the moment is called directly in the init file.

Cheers,
Luca





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21  9:22     ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2020-02-21 14:06       ` Roland Winkler
  2020-02-21 14:08         ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 15+ messages in thread
From: Roland Winkler @ 2020-02-21 14:06 UTC (permalink / raw)
  To: gojjoe2; +Cc: 39686

On Fri Feb 21 2020 gojjoe2@googlemail.com wrote:
> Yes it does, at least for the specific problem I found with
> "\oe". Thank you very much!

Thanks for for confirming it fixes your problem.

> Can you tell me where to insert this redefinition in my init file,
> while I wait for the next Emacs version? I tried adding it to my
> own bibtex-hook and to "eval-after-load 'bibtex ...", but it
> doesn't seem to have any effect in either. At the moment is called
> directly in the init file.

Bibtex-mode uses the variable bibtex-autokey-transcriptions to
define the default values of `bibtex-autokey-name-change-strings'
and `bibtex-autokey-titleword-change-strings'.  So something like
eval-after-load will work only if it deals with all three variables.
Say, you first set bibtex-autokey-transcriptions to its new value;
then you use this to set `bibtex-autokey-name-change-strings' and
`bibtex-autokey-titleword-change-strings'.

It is probably easier to simply set bibtex-autokey-transcriptions in
your init file to its new value.  Normally, emacs should load
bibtex-mode later.  Then it will use your value of
bibtex-autokey-transcriptions to set the values of
`bibtex-autokey-name-change-strings' and
`bibtex-autokey-titleword-change-strings'.





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 14:06       ` Roland Winkler
@ 2020-02-21 14:08         ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 0 replies; 15+ messages in thread
From: gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2020-02-21 14:08 UTC (permalink / raw)
  To: Roland Winkler; +Cc: 39686


> Bibtex-mode uses the variable bibtex-autokey-transcriptions to
> define the default values of `bibtex-autokey-name-change-strings'
> and `bibtex-autokey-titleword-change-strings'.  So something like
> eval-after-load will work only if it deals with all three variables.
> Say, you first set bibtex-autokey-transcriptions to its new value;
> then you use this to set `bibtex-autokey-name-change-strings' and
> `bibtex-autokey-titleword-change-strings'.
> 
> It is probably easier to simply set bibtex-autokey-transcriptions in
> your init file to its new value.  Normally, emacs should load
> bibtex-mode later.  Then it will use your value of
> bibtex-autokey-transcriptions to set the values of
> `bibtex-autokey-name-change-strings' and
> `bibtex-autokey-titleword-change-strings'> 

Thank you for the explanation. I'm doing as you say and everything seems to work smoothly. Please feel free to close this bug report.

Cheers,
Luca





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-20  5:04   ` Roland Winkler
  2020-02-21  9:22     ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2020-02-21 14:13     ` Roland Winkler
  2020-02-21 15:01       ` Eli Zaretskii
  1 sibling, 1 reply; 15+ messages in thread
From: Roland Winkler @ 2020-02-21 14:13 UTC (permalink / raw)
  To: 39686, Eli Zaretskii; +Cc: gojjoe2

Hi Eli

I do not know the exact status of Emacs 27.  Is it OK to install the
patch below into the Emacs 27 branch?  The patch fixes a bug that
must have existed for at least about 20 years.  So likely it will
not affect many users.  But even if the patch has a hidden new bug,
it should not break anything crucial for users either.

Roland



(defvar bibtex-autokey-transcriptions
  (nconc
   (mapcar (lambda (a) (cons (regexp-opt (car a)) (cdr a)))
           '(;; language specific characters
             (("\\aa") . "a")                      ; \aa           -> a
             (("\\AA") . "A")                      ; \AA           -> A
             (("\"a" "\\\"a" "\\ae") . "ae")       ; "a,\"a,\ae    -> ae
             (("\"A" "\\\"A" "\\AE") . "Ae")       ; "A,\"A,\AE    -> Ae
             (("\\i") . "i")                       ; \i            -> i
             (("\\j") . "j")                       ; \j            -> j
             (("\\l") . "l")                       ; \l            -> l
             (("\\L") . "L")                       ; \L            -> L
             (("\"o" "\\\"o" "\\o" "\\oe") . "oe") ; "o,\"o,\o,\oe -> oe
             (("\"O" "\\\"O" "\\O" "\\OE") . "Oe") ; "O,\"O,\O,\OE -> Oe
             (("\"s" "\\\"s" "\\3") . "ss")        ; "s,\"s,\3     -> ss
             (("\"u" "\\\"u") . "ue")              ; "u,\"u        -> ue
             (("\"U" "\\\"U") . "Ue")              ; "U,\"U        -> Ue
             ;; hyphen, accents
             (("\\-" "\\`" "\\'" "\\^" "\\~" "\\=" "\\." "\\u" "\\v"
               "\\H" "\\t" "\\c" "\\d" "\\b") . "")
             ;; space
             (("~") . " ")))
   ;; more spaces
   '(("[\s\t\n]*\\(?:\\\\\\)?[\s\t\n]+" . " ")
     ;; braces, quotes, concatenation.
     ("[`'\"{}#]" . "")))
  "Alist of (OLD-REGEXP . NEW-STRING) pairs.
Used by the default values of `bibtex-autokey-name-change-strings' and
`bibtex-autokey-titleword-change-strings'.  Defaults to translating some
language specific characters to their ASCII transcriptions, and
removing any character accents.")





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 14:13     ` Roland Winkler
@ 2020-02-21 15:01       ` Eli Zaretskii
  2020-02-21 19:50         ` Roland Winkler
  0 siblings, 1 reply; 15+ messages in thread
From: Eli Zaretskii @ 2020-02-21 15:01 UTC (permalink / raw)
  To: Roland Winkler; +Cc: gojjoe2, 39686

> Date: Fri, 21 Feb 2020 08:13:51 -0600
> From: "Roland Winkler" <winkler@gnu.org>
> CC: gojjoe2@googlemail.com
> 
> I do not know the exact status of Emacs 27.  Is it OK to install the
> patch below into the Emacs 27 branch?

I don't see a patch, just a defvar.  Which part(s) of that are
modified, and how?

Also, what commands/features will be affected by this change?

Thanks.





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-20  4:54 bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings Roland Winkler
       [not found] ` <handler.39686.B.158217451717286.ack@debbugs.gnu.org>
@ 2020-02-21 16:05 ` Mattias Engdegård
  2020-02-21 20:03   ` Roland Winkler
  1 sibling, 1 reply; 15+ messages in thread
From: Mattias Engdegård @ 2020-02-21 16:05 UTC (permalink / raw)
  To: Roland Winkler; +Cc: gojjoe2, 39686

Correct me if I'm wrong, but it looks like the assumption is that all possible \-sequences have been enumerated in that list. More likely, some kind of anchoring after the match is needed, like word-end ("\\>"). Then the matching order doesn't matter.






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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 15:01       ` Eli Zaretskii
@ 2020-02-21 19:50         ` Roland Winkler
  2020-02-22  9:12           ` Eli Zaretskii
  0 siblings, 1 reply; 15+ messages in thread
From: Roland Winkler @ 2020-02-21 19:50 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: gojjoe2, 39686

On Fri Feb 21 2020 Eli Zaretskii wrote:
> I don't see a patch, just a defvar.  Which part(s) of that are
> modified, and how?
> 
> Also, what commands/features will be affected by this change?

I am sorry, I had tried to keep things readable for the OP.
A proper patch is attached below.

The patch affects the autokey machinery of bibtex-mode that
calculates keys for new BibTeX entries based on the content of an
entry.  The entry point for the autokey machinery is the function
bibtex-generate-autokey.  In bibtex.el, this function is called once
by the user command bibtex-clean-entry.  No other package inside
emacs relies on this.


diff --git a/lisp/textmodes/bibtex.el b/lisp/textmodes/bibtex.el
index a7be57e..670e763 100644
--- a/lisp/textmodes/bibtex.el
+++ b/lisp/textmodes/bibtex.el
@@ -1006,32 +1006,36 @@ bibtex-autokey-expand-strings
   :type 'boolean)
 
 (defvar bibtex-autokey-transcriptions
-  '(;; language specific characters
-    ("\\\\aa" . "a")                      ; \aa           -> a
-    ("\\\\AA" . "A")                      ; \AA           -> A
-    ("\\\"a\\|\\\\\\\"a\\|\\\\ae" . "ae") ; "a,\"a,\ae    -> ae
-    ("\\\"A\\|\\\\\\\"A\\|\\\\AE" . "Ae") ; "A,\"A,\AE    -> Ae
-    ("\\\\i" . "i")                       ; \i            -> i
-    ("\\\\j" . "j")                       ; \j            -> j
-    ("\\\\l" . "l")                       ; \l            -> l
-    ("\\\\L" . "L")                       ; \L            -> L
-    ("\\\"o\\|\\\\\\\"o\\|\\\\o\\|\\\\oe" . "oe") ; "o,\"o,\o,\oe -> oe
-    ("\\\"O\\|\\\\\\\"O\\|\\\\O\\|\\\\OE" . "Oe") ; "O,\"O,\O,\OE -> Oe
-    ("\\\"s\\|\\\\\\\"s\\|\\\\3" . "ss")  ; "s,\"s,\3     -> ss
-    ("\\\"u\\|\\\\\\\"u" . "ue")          ; "u,\"u        -> ue
-    ("\\\"U\\|\\\\\\\"U" . "Ue")          ; "U,\"U        -> Ue
-    ;; accents
-    ("\\\\`\\|\\\\'\\|\\\\\\^\\|\\\\~\\|\\\\=\\|\\\\\\.\\|\\\\u\\|\\\\v\\|\\\\H\\|\\\\t\\|\\\\c\\|\\\\d\\|\\\\b" . "")
-    ;; braces, quotes, concatenation.
-    ("[`'\"{}#]" . "")
-    ("\\\\-" . "")                        ; \-            ->
-    ;; spaces
-    ("\\\\?[ \t\n]+\\|~" . " "))
+  (nconc
+   (mapcar (lambda (a) (cons (regexp-opt (car a)) (cdr a)))
+           '(;; language specific characters
+             (("\\aa") . "a")                      ; \aa           -> a
+             (("\\AA") . "A")                      ; \AA           -> A
+             (("\"a" "\\\"a" "\\ae") . "ae")       ; "a,\"a,\ae    -> ae
+             (("\"A" "\\\"A" "\\AE") . "Ae")       ; "A,\"A,\AE    -> Ae
+             (("\\i") . "i")                       ; \i            -> i
+             (("\\j") . "j")                       ; \j            -> j
+             (("\\l") . "l")                       ; \l            -> l
+             (("\\L") . "L")                       ; \L            -> L
+             (("\"o" "\\\"o" "\\o" "\\oe") . "oe") ; "o,\"o,\o,\oe -> oe
+             (("\"O" "\\\"O" "\\O" "\\OE") . "Oe") ; "O,\"O,\O,\OE -> Oe
+             (("\"s" "\\\"s" "\\3") . "ss")        ; "s,\"s,\3     -> ss
+             (("\"u" "\\\"u") . "ue")              ; "u,\"u        -> ue
+             (("\"U" "\\\"U") . "Ue")              ; "U,\"U        -> Ue
+             ;; hyphen, accents
+             (("\\-" "\\`" "\\'" "\\^" "\\~" "\\=" "\\." "\\u" "\\v"
+               "\\H" "\\t" "\\c" "\\d" "\\b") . "")
+             ;; space
+             (("~") . " ")))
+   ;; more spaces
+   '(("[\s\t\n]*\\(?:\\\\\\)?[\s\t\n]+" . " ")
+     ;; braces, quotes, concatenation.
+     ("[`'\"{}#]" . "")))
   "Alist of (OLD-REGEXP . NEW-STRING) pairs.
-Used by the default values of `bibtex-autokey-name-change-strings' and
+Used as default values of `bibtex-autokey-name-change-strings' and
 `bibtex-autokey-titleword-change-strings'.  Defaults to translating some
 language specific characters to their ASCII transcriptions, and
-removing any character accents.")
+removing any accent characters.")
 
 (defcustom bibtex-autokey-name-change-strings
   bibtex-autokey-transcriptions





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 16:05 ` Mattias Engdegård
@ 2020-02-21 20:03   ` Roland Winkler
  2020-02-21 20:43     ` Mattias Engdegård
  0 siblings, 1 reply; 15+ messages in thread
From: Roland Winkler @ 2020-02-21 20:03 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: gojjoe2, 39686

On Fri Feb 21 2020 Mattias Engdegård wrote:
> Correct me if I'm wrong, but it looks like the assumption is that
> all possible \-sequences have been enumerated in that list. More
> likely, some kind of anchoring after the match is needed, like
> word-end ("\\>"). Then the matching order doesn't matter.

I am not sure I understand.  The idea in bibtex-autokey-transcriptions
is that each match of OLD-REGEXP is replaced in full by NEW-STRING.
Subexpressions are ignored.

In LaTeX syntax, OLD-REGEXP can appear anywhere inside what LaTeX
considers a word (which even may include spaces).  So to make things
more fool-proofed, it would be necessary to parse more carefully the
LaTeX code.  I do not think this effort is needed here as these
regexps have worked well for at least two decades.  The patch fixes
a minor problem of these regexps pointed out by the OP.  But
otherwise it preserves their spirit.





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 20:03   ` Roland Winkler
@ 2020-02-21 20:43     ` Mattias Engdegård
  2020-02-21 21:03       ` Roland Winkler
  0 siblings, 1 reply; 15+ messages in thread
From: Mattias Engdegård @ 2020-02-21 20:43 UTC (permalink / raw)
  To: Roland Winkler; +Cc: gojjoe2, 39686

21 feb. 2020 kl. 21.03 skrev Roland Winkler <winkler@gnu.org>:

> In LaTeX syntax, OLD-REGEXP can appear anywhere inside what LaTeX
> considers a word (which even may include spaces).  So to make things
> more fool-proofed, it would be necessary to parse more carefully the
> LaTeX code.  I do not think this effort is needed here as these
> regexps have worked well for at least two decades.  The patch fixes
> a minor problem of these regexps pointed out by the OP.  But
> otherwise it preserves their spirit.

In LaTeX you can't just write 'b\oeuf'; it will complain that '\oeuf' is undefined. You have to write 'b\oe uf' or 'b{\oe}uf'. Thus there is a word break at the end. (Accents, like '\"o', are different; there is only a single letter after the '\"'.)
With your table, you replace '\o' with 'oe', but what if the text uses a different \-sequence starting with \o, like '\omega'? After substitution, you would have 'oemega' which wasn't intended.

Safer then to tack on a zero-width assertion, like

"\\\\\\(?:o\\|oe\\)\\>"

for example. Or, if you think it's hard to read,

(rx "\\" (or "o" "oe") word-end)







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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 20:43     ` Mattias Engdegård
@ 2020-02-21 21:03       ` Roland Winkler
  2020-02-21 21:32         ` Mattias Engdegård
  0 siblings, 1 reply; 15+ messages in thread
From: Roland Winkler @ 2020-02-21 21:03 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: gojjoe2, 39686

On Fri Feb 21 2020 Mattias Engdegård wrote:
> In LaTeX you can't just write 'b\oeuf'; it will complain that
> '\oeuf' is undefined. You have to write 'b\oe uf' or
> 'b{\oe}uf'. Thus there is a word break at the end.

In bibtex-mode, all this is used in the context of what is supposed
to represent names and titles of books and articles.  The current
scheme has been in place for at least two decades and nobody
complained about this.  (Note that the current thread is about
something else.)  So I assume that what you are concerned about is
so rare (at least in the context of the autokey machinery) that it
is not worth the effort.

But feel free to provide a patch along these lines.  However, that
patch should certainly go into master because it modifies the
current scheme.





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 21:03       ` Roland Winkler
@ 2020-02-21 21:32         ` Mattias Engdegård
  0 siblings, 0 replies; 15+ messages in thread
From: Mattias Engdegård @ 2020-02-21 21:32 UTC (permalink / raw)
  To: Roland Winkler; +Cc: gojjoe2, 39686

21 feb. 2020 kl. 22.03 skrev Roland Winkler <winkler@gnu.org>:

> So I assume that what you are concerned about is
> so rare (at least in the context of the autokey machinery) that it
> is not worth the effort.

Fair enough. If you run into such a partial match problem in the future, you now know at least one more way to solve it.






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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-21 19:50         ` Roland Winkler
@ 2020-02-22  9:12           ` Eli Zaretskii
  2020-03-06  8:46             ` Roland Winkler
  0 siblings, 1 reply; 15+ messages in thread
From: Eli Zaretskii @ 2020-02-22  9:12 UTC (permalink / raw)
  To: Roland Winkler; +Cc: gojjoe2, 39686

> Date: Fri, 21 Feb 2020 13:50:01 -0600
> From: "Roland Winkler" <winkler@gnu.org>
> Cc: 39686@debbugs.gnu.org,
>     gojjoe2@googlemail.com
> 
> On Fri Feb 21 2020 Eli Zaretskii wrote:
> > I don't see a patch, just a defvar.  Which part(s) of that are
> > modified, and how?
> > 
> > Also, what commands/features will be affected by this change?
> 
> I am sorry, I had tried to keep things readable for the OP.
> A proper patch is attached below.
> 
> The patch affects the autokey machinery of bibtex-mode that
> calculates keys for new BibTeX entries based on the content of an
> entry.  The entry point for the autokey machinery is the function
> bibtex-generate-autokey.  In bibtex.el, this function is called once
> by the user command bibtex-clean-entry.  No other package inside
> emacs relies on this.

Thanks.  I'm fine with installing this on the emacs-27 branch.





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

* bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings
  2020-02-22  9:12           ` Eli Zaretskii
@ 2020-03-06  8:46             ` Roland Winkler
  0 siblings, 0 replies; 15+ messages in thread
From: Roland Winkler @ 2020-03-06  8:46 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 39686-done, gojjoe2

On Sat Feb 22 2020 Eli Zaretskii wrote:
> > The patch affects the autokey machinery of bibtex-mode that
> > calculates keys for new BibTeX entries based on the content of an
> > entry.  The entry point for the autokey machinery is the function
> > bibtex-generate-autokey.  In bibtex.el, this function is called once
> > by the user command bibtex-clean-entry.  No other package inside
> > emacs relies on this.
> 
> Thanks.  I'm fine with installing this on the emacs-27 branch.

Done (commit cb1877321b8a04cdb9b890d76d99a9f5a7ed5bce).

I am sorry for the delay.





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

end of thread, other threads:[~2020-03-06  8:46 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-02-20  4:54 bug#39686: 25.2; Wrong behaviour of bibtex-autokey-name-change-strings Roland Winkler
     [not found] ` <handler.39686.B.158217451717286.ack@debbugs.gnu.org>
2020-02-20  5:04   ` Roland Winkler
2020-02-21  9:22     ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
2020-02-21 14:06       ` Roland Winkler
2020-02-21 14:08         ` gojjoe2--- via Bug reports for GNU Emacs, the Swiss army knife of text editors
2020-02-21 14:13     ` Roland Winkler
2020-02-21 15:01       ` Eli Zaretskii
2020-02-21 19:50         ` Roland Winkler
2020-02-22  9:12           ` Eli Zaretskii
2020-03-06  8:46             ` Roland Winkler
2020-02-21 16:05 ` Mattias Engdegård
2020-02-21 20:03   ` Roland Winkler
2020-02-21 20:43     ` Mattias Engdegård
2020-02-21 21:03       ` Roland Winkler
2020-02-21 21:32         ` Mattias Engdegård

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).