all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Eli Zaretskii'" <eliz@gnu.org>
Cc: 12872@debbugs.gnu.org
Subject: bug#12872: bug#12867: 24.3.50; easy-to-repro crash involving mode line
Date: Mon, 12 Nov 2012 11:07:11 -0800	[thread overview]
Message-ID: <EFC709B8B50148D7A9F495F3F2E9F783@us.oracle.com> (raw)
In-Reply-To: <83625awtyj.fsf@gnu.org>

[from bug #12867]

> I mean a new option, an enhancement.
> 
> > > But we should first formulate the conditions under which this
> > > redisplay will be performed.
> > 
> > If we're talking about my use case then it is each time the 
> > current line changes.
> 
> Would it be good enough to redisplay whenever point moves, and let
> your code you run from :eval decide whether the text on the mode line
> needs to be changed?  I think this will be a more general solution.

Yes, it would be good enough.

But the advantage that I'm supposing %l has is that the line-counting is done in
C, as part of the display engine.

If my code had to check whether the line has changed then it would do that in
Lisp.  Not saying that's a big deal.  But it still looks to me like the %l
triggering is convenient.

Perhaps the option could handle both cases: the general point-change case and
the more particular line-change case, depending on the option value?

BTW, why would this be a user option, rather than just a variable that code can
bind?  The use case for users is not too clear to me.

I guess you want users to be able to turn off such triggering?  That is
something different from turning off redisplay caused by such triggering (of
course, inhibiting the triggering turns off its resulting redisplay also).

Anyway, I don't have much to say about what should be done for this enhancement.

> Done: bug #12872.  Let's continue there.

Thank you.






  reply	other threads:[~2012-11-12 19:07 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-12  3:52 bug#12867: 24.3.50; easy-to-repro crash involving mode line Drew Adams
2012-11-12 15:40 ` Eli Zaretskii
2012-11-12 17:09   ` Drew Adams
2012-11-12 17:52     ` Eli Zaretskii
2012-11-12 18:16       ` Drew Adams
2012-11-12 18:35         ` Eli Zaretskii
2012-11-12 19:07           ` Drew Adams [this message]
2012-11-12 17:31   ` martin rudalics
2012-11-12 18:19     ` Eli Zaretskii
2012-11-12 20:55       ` Stefan Monnier
2012-11-13  8:22         ` martin rudalics
2012-11-13 12:52           ` Eli Zaretskii
2012-11-13 13:50             ` martin rudalics

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=EFC709B8B50148D7A9F495F3F2E9F783@us.oracle.com \
    --to=drew.adams@oracle.com \
    --cc=12872@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.