Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

bug#5175: marked as done (23.1.50; mark deactivated without running deactivate-mark-hook when replying with Gnus)

0 views
Skip to first unread message

Emacs bug Tracking System

unread,
Jan 1, 2010, 4:29:02 PM1/1/10
to Chong Yidong, emacs-bu...@debbugs.gnu.org
Your message dated Fri, 01 Jan 2010 15:43:29 -0500
with message-id <87r5q93...@stupidchicken.com>
and subject line 23.1.50; mark deactivated without running deactivate-mark-hook when replying with Gnus
has caused the Emacs bug report #5175,
regarding 23.1.50; mark deactivated without running deactivate-mark-hook when replying with Gnus
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact bug-gn...@gnu.org
immediately.)


--
5175: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=5175
Emacs Bug Tracking System
Contact bug-gn...@gnu.org with problems

Katsumi Yamaoka

unread,
Jan 1, 2010, 8:38:12 PM1/1/10
to Chong Yidong, emacs-pr...@gnu.org, Tassilo Horn, 51...@emacsbugs.donarmstrong.com, di...@gnus.org
>>>>> In <87aaxqd...@thinkpad.tsdh.de>
>>>>> Tassilo Horn <tas...@member.fsf.org> wrote:
> I use this code, to have a 'bar cursor type when the mark is active.

> (defun th-activate-mark-init ()
> (setq cursor-type 'bar))
> (add-hook 'activate-mark-hook 'th-activate-mark-init)

> (defun th-deactivate-mark-init ()
> (setq cursor-type 'box))
> (add-hook 'deactivate-mark-hook 'th-deactivate-mark-init)

> This works very good, except when replying to a mail with Gnus using `R'
> in summary. `R' first yanks the quoted contents of the email I want to
> reply to into the message buffer. After that, I have a 'bar cursor but
> no active region.

> So it seems that it's somehow possible to deactivate the mark without
> running deactivate-mark-hook, and I think that shouldn't be so.

>>>>> In Bzr-99227 Chong Yidong wrote:
> 2010-01-01 Chong Yidong <c...@stupidchicken.com>

> * message.el (message-exchange-point-and-mark): Call
> exchange-point-and-mark with an argument rather than setting
> mark-active by hand (Bug#5175).

This incompatible change makes Gnus not work with old Emacsen, so
I don't merge it to the Gnus trunk, i.e., No Gnus. Any idea not
to use the argument of `exchange-point-and-mark'?


Katsumi Yamaoka

unread,
Jan 1, 2010, 8:38:12 PM1/1/10
to Chong Yidong, emacs-pr...@gnu.org, Tassilo Horn, 51...@emacsbugs.donarmstrong.com, di...@gnus.org

Chong Yidong

unread,
Jan 1, 2010, 9:41:59 PM1/1/10
to Katsumi Yamaoka, Tassilo Horn, 51...@debbugs.gnu.org, di...@gnus.org
Katsumi Yamaoka <yam...@jpl.org> writes:

> This incompatible change makes Gnus not work with old Emacsen, so
> I don't merge it to the Gnus trunk, i.e., No Gnus. Any idea not
> to use the argument of `exchange-point-and-mark'?

An alternative is to avoid using exchange-point-and-mark entirely, like
this:

(goto-char (prog1 (mark t)
(set-marker (mark-marker) (point)
(current-buffer))))

(Please don't replay to emacsbugs.donarmstrong.com/debbugs.gnu.org AND
emacs-pretest-bug at the same time; sending to the latter creates a new
bug in our database.)


Chong Yidong

unread,
Jan 2, 2010, 2:20:26 PM1/2/10
to Katsumi Yamaoka, Tassilo Horn, 51...@debbugs.gnu.org, di...@gnus.org
Katsumi Yamaoka <yam...@jpl.org> writes:

>> * message.el (message-exchange-point-and-mark): Call
>> exchange-point-and-mark with an argument rather than setting
>> mark-active by hand (Bug#5175).
>

> This incompatible change makes Gnus not work with old Emacsen, so
> I don't merge it to the Gnus trunk, i.e., No Gnus. Any idea not
> to use the argument of `exchange-point-and-mark'?

I've reworked the change to use the simpler functions from my previous
message; I think this should solve the backward compatibility problem.


Katsumi Yamaoka

unread,
Jan 4, 2010, 9:23:54 PM1/4/10
to Chong Yidong, Tassilo Horn, 51...@debbugs.gnu.org, di...@gnus.org

Thanks for fixing it. I've verified it also clears the problem:

http://news.gmane.org/group/gmane.emacs.gnus.general/thread=56821


0 new messages