Eli Zaretskii <eliz@gnu.org> writes:

>> +    {
>> +      /* We haven't implemented the code to dump overlays.  */
>> +      error ("Dumping overlays in buffers is not yet implemented.  Aborting...\n");
>> +      emacs_abort ();
>> +    }
>
> If you call 'error' in batch mode, do you really need emacs_abort?

If `error' does all the necessary cleanup, `emacs_abort' is indeed not
necessary. But that's not very obvious from the source code.

See the attached patch with `emacs_abort` call removed.
I also removed the final newline, as it does not seem to be used in
other instances of error (...) calls.