all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Dmitry Antipov <dmantipov@yandex.ru>
Cc: 12426@debbugs.gnu.org, rms@gnu.org
Subject: bug#12426: 24.2.50; Emacs is closed unexpectedly after query-replace
Date: Fri, 14 Sep 2012 16:40:48 +0300	[thread overview]
Message-ID: <834nn0vi1r.fsf@gnu.org> (raw)
In-Reply-To: <5053247C.9050708@yandex.ru>

> Date: Fri, 14 Sep 2012 16:35:08 +0400
> From: Dmitry Antipov <dmantipov@yandex.ru>
> CC: Richard Stallman <rms@gnu.org>, dmoncayo@gmail.com, 
>  12426@debbugs.gnu.org
> 
> On 09/13/2012 08:47 PM, Eli Zaretskii wrote:
> 
> > But marker_position and marker_byte_position are simple getters of
> > these two attributes of a marker.  If these attributes can be out of
> > range for some window of time, then the getters shouldn't enforce this
> > limitation.  Otherwise, they are getters that cannot be used in some
> > situations, which is IMO bad SE.  At the very least that should be
> > documented.
> 
> IIUC no, since this window of time is very short and it's entirely
> within adjust_markers_for_* functions.

The length of the window has no importance whatsoever.  It just takes
a couple of machine instructions to trigger the assertion violation.

This isn't a stock exchange or a poker game, where we could build on
chances.  This is software that is supposed to be stable, and you are
talking about its most inner and intimate internals.

If you can figure out how to avoid the crash, please do.  Otherwise,
please remove the eassert calls, because they make Emacs completely
unusable.





  reply	other threads:[~2012-09-14 13:40 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-12 18:06 bug#12426: 24.2.50; Emacs is closed unexpectedly after query-replace Dani Moncayo
2012-09-12 18:39 ` Eli Zaretskii
2012-09-12 19:02   ` Eli Zaretskii
2012-09-13  3:01     ` Dmitry Antipov
2012-09-13 16:47       ` Eli Zaretskii
2012-09-14 12:35         ` Dmitry Antipov
2012-09-14 13:40           ` Eli Zaretskii [this message]
2012-09-17 17:51             ` Dani Moncayo
2012-09-12 19:15   ` Eli Zaretskii

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=834nn0vi1r.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=12426@debbugs.gnu.org \
    --cc=dmantipov@yandex.ru \
    --cc=rms@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.