This bug looks a little weird to me, because I can not find any changes to lwarn in the Emacs commit history nor in the sx.el commit history. However, Emacs expects the first argument of lwarn to be a symbol or a list (and currently errors because the first argument is a string, so that sx.el basic functionality is broken). This commit fixes that.
I am not sure how lwarn used to work with strings as I can not easily find any information about that, but I just converted the string into a list which seems appropriate to me (from reading the lwarn docstring).
vermiculus
commented
2 years ago
This bug looks a little weird to me, because I can not find any changes to
lwarnin the Emacs commit history nor in the sx.el commit history. However, Emacs expects the first argument of lwarn to be a symbol or a list (and currently errors because the first argument is a string, so that sx.el basic functionality is broken). This commit fixes that.I am not sure how
lwarnused to work with strings as I can not easily find any information about that, but I just converted the string into a list which seems appropriate to me (from reading thelwarndocstring).