unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: emacs-25 ea512a7: * lisp/gnus/mm-decode.el (mm-convert-shr-links):
       [not found] ` <20160622103321.26666220123@vcs.savannah.gnu.org>
@ 2016-07-25 23:50   ` Katsumi Yamaoka
  2016-09-26 22:35     ` Katsumi Yamaoka
  0 siblings, 1 reply; 2+ messages in thread
From: Katsumi Yamaoka @ 2016-07-25 23:50 UTC (permalink / raw)
  To: emacs-devel

On Wed, 22 Jun 2016 10:33:20 +0000, Katsumi Yamaoka wrote:
> branch: emacs-25
> commit ea512a7c2cd30206dd509b193c9faaba25640180
> Author: Katsumi Yamaoka <yamaoka@jpl.org>
> Commit: Katsumi Yamaoka <yamaoka@jpl.org>

>     * lisp/gnus/mm-decode.el (mm-convert-shr-links):
>     Mask keys that launch `widget-button-click' (bug#22157).
[...]
> -	(put-text-property start end 'local-map nil)
> +	;; Remove keymap that `shr-urlify' adds.
> +	(put-text-property start end 'keymap nil)
...[1]

This was done due to my having misunderstood the old code.  So,
I'd like to backport the fix (already done in master [2]) to
emacs-25.  May I do it?

The purpose of the old code seems to be to make void a spurious
`local-map' that something other might have added.  My fault is
that I thought `local-map' was a just a misspelling of `keymap'.
The key bindings that shr adds must be preserved anyway [3].

[1] <http://lists.gnu.org/archive/html/emacs-diffs/2016-06/msg00173.html>
[2] <http://article.gmane.org/gmane.emacs.diffs/136469>
[3] <http://lists.gnu.org/archive/html/bug-gnu-emacs/2016-07/msg00729.html>

Regards,



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

* Re: emacs-25 ea512a7: * lisp/gnus/mm-decode.el (mm-convert-shr-links):
  2016-07-25 23:50   ` emacs-25 ea512a7: * lisp/gnus/mm-decode.el (mm-convert-shr-links): Katsumi Yamaoka
@ 2016-09-26 22:35     ` Katsumi Yamaoka
  0 siblings, 0 replies; 2+ messages in thread
From: Katsumi Yamaoka @ 2016-09-26 22:35 UTC (permalink / raw)
  To: emacs-devel

On Tue, 26 Jul 2016 08:50:13 +0900, Katsumi Yamaoka wrote:
> On Wed, 22 Jun 2016 10:33:20 +0000, Katsumi Yamaoka wrote:
>> branch: emacs-25
>> commit ea512a7c2cd30206dd509b193c9faaba25640180
>> Author: Katsumi Yamaoka <yamaoka@jpl.org>
>> Commit: Katsumi Yamaoka <yamaoka@jpl.org>

>>     * lisp/gnus/mm-decode.el (mm-convert-shr-links):
>>     Mask keys that launch `widget-button-click' (bug#22157).
> [...]
>> -	(put-text-property start end 'local-map nil)
>> +	;; Remove keymap that `shr-urlify' adds.
>> +	(put-text-property start end 'keymap nil)
> ...[1]

> This was done due to my having misunderstood the old code.  So,
> I'd like to backport the fix (already done in master [2]) to
> emacs-25.  May I do it?

I've backported the fix from master.

> The purpose of the old code seems to be to make void a spurious
> `local-map' that something other might have added.  My fault is
> that I thought `local-map' was a just a misspelling of `keymap'.
> The key bindings that shr adds must be preserved anyway [3].

> [1] <http://lists.gnu.org/archive/html/emacs-diffs/2016-06/msg00173.html>
> [2] <http://article.gmane.org/gmane.emacs.diffs/136469>
> [3] <http://lists.gnu.org/archive/html/bug-gnu-emacs/2016-07/msg00729.html>

Regards,



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

end of thread, other threads:[~2016-09-26 22:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20160622103320.1424.37554@vcs.savannah.gnu.org>
     [not found] ` <20160622103321.26666220123@vcs.savannah.gnu.org>
2016-07-25 23:50   ` emacs-25 ea512a7: * lisp/gnus/mm-decode.el (mm-convert-shr-links): Katsumi Yamaoka
2016-09-26 22:35     ` Katsumi Yamaoka

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