Use xref-push-marker-stack when available

joaotavora / sly

Sylvester the Cat's Common Lisp IDE

1.26k stars 142 forks source link

Closed theothornhill closed 2 years ago

theothornhill commented 2 years ago

sly.el (sly-push-definition-stack): Inserting directly into find-tag-marker-ring is obsolete since emacs 25.1. As of emacs 29 sly no longer is able to jump back and forth between definition and usage with M-. and M-,. As a precaution for backwards compatibility, we use xref-push-marker-stack only when it is bound.

theothornhill commented 2 years ago

Closing since there already is an issue #479

joaotavora commented 2 years ago

Yes, but you PR seems more green-buttonable :-)

theothornhill commented 2 years ago

Great! Thanks!