Re: undo refactoring

[Stefan Monnier <monnier@iro.umontreal.ca>]

> Why not just do

> 	bset_undo_list (current_buffer,
> 			Fcons (Qnil, BVAR (current_buffer, undo_list)));

> and remove all references to pending_boundary (and therefore
> `prepare_record` which is now rather badly named)? Or am I missing
> something

IIUC, the reason for this was so as to avoid an inf-loop when malloc
fails (for lack of memory).

The particular issue was that it was OK for a command to fail because of
malloc failure, but it was not OK for the read-eval-loop itself to fail
before having the opportunity to run another command.

My impression is that this design goal has been ignored for much too
long, so I don't think Emacs behaves this way any more.  And instead we
have now other ways to handle the memory-full situation (such as the
extra memory that's pre-allocated and then released in case the memory
is full).

IOW, I'd be surprised if getting rid of this quirk would ever lead to
a visible change in Emacs's behavior nowadays.


        Stefan