bug#66743: 30.0.50; Crash when dumping reftex

[Eli Zaretskii]

> From: Ihor Radchenko <yantar92@posteo.net>
> Cc: gerd.moellmann@gmail.com, stefankangas@gmail.com, 66743@debbugs.gnu.org
> Date: Fri, 27 Oct 2023 10:55:37 +0000
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> If `error' does all the necessary cleanup, `emacs_abort' is indeed not
> >> necessary. But that's not very obvious from the source code.
> >
> > What cleanup is needed in this case?
> 
> I have no idea.
> To be clear: I am not sure what is the difference between `emacs_abort'
> and `error'.

AFAIU, in batch mode there's no difference, since calling 'error'
shows a backtrace and exits.

> If you say that using `error' instead of `emacs_abort' is safe, I will
> just follow. I expect that you know these details well.

I think it's safe, since we already do that in other places in
pdumper.c.