unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Richard Stallman <rms@gnu.org>
To: "Kalman Reti" <kalman.reti@gmail.com>
Cc: bug-gnu-emacs@gnu.org, kalman.reti@gmail.com,
	monnier@iro.umontreal.ca, emacs-devel@gnu.org
Subject: Re: mark_object crash in 22.1 and latest CVS (as of tonight)
Date: Wed, 14 Nov 2007 22:08:23 -0500	[thread overview]
Message-ID: <E1IsV5b-0004Vn-Na@fencepost.gnu.org> (raw)
In-Reply-To: <16af2f430711140939x45663644je0dce25c8796b18@mail.gmail.com> (kalman.reti@gmail.com)

Nothing gets "removed" from the undo list in normal use.  It gets
truncated, which drops off elements at the end, but other than that
all that normally happens is that editing operations add elements.

Markers in the list should not become free, because the undo list
itself should preserve them from GC.

If this is reproducible, can you put a breakpoint at Fgarbage_collect
and examine the data just before the GC which gets this crash?
Examine that list using the x... commands, and see if that marker
is already free.

    Looking harder at the code, I'm convinced that the undo_list should come before
    the name entry in the buffer structure,

Definitely not.  It needs to be AFTER `name' so that it will be marked
by GC.

    Anyone know what the elements of the undo_list mean?  Some are conses
    with a marker
    in their CAR and a number in their CDR, some are just conses of two
    numbers and some
    are conses of a string and a number.

The Lisp Manual documents these.  Node `Undo'.





  parent reply	other threads:[~2007-11-15  3:08 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <16af2f430711081955j3d5e6745gc0f7a50e02d9a892@mail.gmail.com>
     [not found] ` <16af2f430711090332x39c8c768gb01d1dc6a0e4f427@mail.gmail.com>
2007-11-11  5:22   ` mark_object crash in 22.1 and latest CVS (as of tonight) Richard Stallman
2007-11-12 11:40     ` Kalman Reti
2007-11-12 22:03       ` Stefan Monnier
2007-11-13  0:30         ` Kalman Reti
2007-11-13 20:03         ` Richard Stallman
2007-11-14 17:39           ` Kalman Reti
2007-11-14 18:51             ` Stefan Monnier
2007-11-15  1:00               ` Kalman Reti
2007-11-15 17:09                 ` Richard Stallman
2007-11-16 12:05                   ` Kalman Reti
2007-11-16 14:07                     ` Kalman Reti
2007-11-16 17:28                       ` martin rudalics
2007-11-16 17:56                         ` Kalman Reti
2007-11-17  4:54                           ` Richard Stallman
2007-11-17  5:43                             ` Kalman Reti
2007-11-16 19:04                         ` Stefan Monnier
2007-11-16 21:52                           ` martin rudalics
2007-11-16 22:09                             ` Stefan Monnier
2007-11-16 22:16                             ` Stefan Monnier
2007-11-16 23:59                               ` Kalman Reti
2007-11-17  4:25                                 ` Stefan Monnier
2007-11-17 17:42                           ` Richard Stallman
2007-11-18  3:08                             ` Stefan Monnier
2007-11-18 22:45                               ` Richard Stallman
2007-11-18 23:22                                 ` David Kastrup
2007-11-19  7:50                                   ` Stefan Monnier
2007-11-19 19:03                                   ` Richard Stallman
2007-11-17  4:53                         ` Richard Stallman
2007-11-15  3:08             ` Richard Stallman [this message]
2007-11-15  8:38               ` Kalman Reti
2007-11-16 20:48                 ` Kalman Reti
2007-11-16 21:59                   ` Stefan Monnier
2007-11-16 23:09                     ` martin rudalics
2008-02-04 11:35                     ` buffer-undo-list (was: mark_object crash in 22.1 and latest CVS (as of tonight)) Johan Bockgård
2008-02-04 21:44                       ` buffer-undo-list Stefan Monnier
2008-02-11 17:58                       ` buffer-undo-list Stefan Monnier
2007-11-13  5:10       ` mark_object crash in 22.1 and latest CVS (as of tonight) Richard Stallman

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=E1IsV5b-0004Vn-Na@fencepost.gnu.org \
    --to=rms@gnu.org \
    --cc=bug-gnu-emacs@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=kalman.reti@gmail.com \
    --cc=monnier@iro.umontreal.ca \
    /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 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).