bug#74274: [PATCH] Revert part of d3f8ed730f to avoid segmentation fault

["Gerd Möllmann" <gerd.moellmann@gmail.com>]

Gerd Möllmann <gerd.moellmann@gmail.com> writes:

> Eli Zaretskii <eliz@gnu.org> writes:
>
>> Should this perhaps have an eassert which verifies that every glyph
>> has a valid frame pointer?  At the very least please add a comment
>> there explaining the need for the enabled_p test and mentioning the
>> frame pointer of the glyphs.
>
> I don't know. It's one of the most basic things about the meaning of
> enabled_p in current glyphs, and it's not limited the frame pointer in
> any way.

Found another case of copying from non-enabled glyphs. Reproducable with
emacs -q -l with a file containing

  (with-current-buffer (get-buffer-create "1")
    (setq header-line-format
          '((:eval (format "*package*: -  symbol-packages: - lexical-binding: %s"
                           lexical-binding)))))
  (with-current-buffer (get-buffer-create "2")
    (insert "something"))

  (let ((w1 (selected-window))
        (w2 (split-window-right)))
    (set-window-buffer w1 (get-buffer "1"))
    (set-window-buffer w2 (get-buffer "2"))
    (message "message"))

It's a 30 years old bug, so it isn't something urgent to fix.

I'll port something to savannah as soon as I find the time.