Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix syntax-table class of carriage returns #680

Merged
merged 2 commits into from
May 16, 2024

Conversation

yuhan0
Copy link
Contributor

@yuhan0 yuhan0 commented May 15, 2024

The syntax table interprets ASCII control chars (including CRs) as symbols in clojure buffers, affecting functions such as (thing-at-point 'symbol)
This caused clojure-find-ns to report "^M" as the namespace in some files with \r\n line encodings.


Before submitting a PR mark the checkboxes for the items you've done (if you
think a checkbox does not apply, then leave it unchecked):

  • The commits are consistent with our contribution guidelines.
  • You've added tests (if possible) to cover your change(s). Bugfix, indentation, and font-lock tests are extremely important!
  • You've run M-x checkdoc and fixed any warnings in the code you've written.
  • You've updated the changelog (if adding/changing user-visible functionality).
  • You've updated the readme (if adding/changing user-visible functionality).

Thanks!

The syntax table interprets ASCII control chars (including CRs) as
symbols in clojure buffers, affecting functions such as
(thing-at-point 'symbol)
This caused clojure-find-ns to report "^M" as the namespace in some
files with \r\n line encodings.
@bbatsov bbatsov merged commit baddab3 into clojure-emacs:master May 16, 2024
7 checks passed
@bbatsov
Copy link
Member

bbatsov commented May 16, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants