emacs-slack icon indicating copy to clipboard operation
emacs-slack copied to clipboard

Fix missing lexbind declaration in slack-user-message.

Open ieure opened this issue 3 years ago • 1 comments

Recent Emacs builds have changed how cl-call-next-method works in a way that causes breakage when used with dynamic binding (but they work fine with lexical binding). The slack-user-message.el file appears to have a missing / broken / incomplete header, which causes it to default to dynamic binding, which causes cl-call-next-method inside `slack-message-body' to fail.

This commit adds the file header and declares that slack-user-message should use lexical binding.

fixes yuya373/emacs-slack#570

ieure avatar Sep 17 '22 17:09 ieure

That fixes it for me, thanks =D

unhammer avatar Sep 18 '22 12:09 unhammer