Re: mail-abbrev-end-of-buffer apparently does nothing

[Richard Stallman]

    > +                       ;; Do let Ctl and Meta chars expand if they try.
    > +                       (not (event-modifiers last-command-char))
    > +                       (equal this-command 'self-insert)

    Why check (not (event-modifiers last-command-char)), then?

Maybe it isn't necessary given the other condition.

But the main question is, does this code fully work right?