Skip to content

Syntax table entries for non-alphanumeric chars #412

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

Merged
merged 1 commit into from
Jan 1, 2015

Conversation

ikirill
Copy link
Contributor

@ikirill ikirill commented Dec 31, 2014

See #410

List of characters generated with the unicode-fonts package

(mapc
 (lambda (block)
   (when (and (string-match-p "operators\\|punctuation\\|symbol\\|arrow"
                              (car block))
              (not (string-match-p "alpha\\|letterlike" (car block))))
     (insert
      (format "\n%-45s ; %s"
              (prin1-to-string
               `(modify-syntax-entry (cons ,(nth 1 block) ,(nth 2 block)) "_" table))
              (car block)))))
 unicode-fonts-blocks)

I hope I didn't miss any important characters.

@purcell
Copy link
Member

purcell commented Dec 31, 2014

Excellent, thanks!

  • Do we know if this is the same list of ranges Haskell itself is using?
  • Can you check if can now be removed from the string of characters above? If so, you can amend the commit and force-push it to update the PR.
  • mapc over a list would perhaps be a little cleaner than repeatedly calling modify-syntax-entry.

@ikirill ikirill force-pushed the unicode-symbol-syntax branch from c6bb545 to d2a931a Compare December 31, 2014 21:24
@ikirill
Copy link
Contributor Author

ikirill commented Dec 31, 2014

purcell added a commit that referenced this pull request Jan 1, 2015
Syntax table entries for non-alphanumeric chars
@purcell purcell merged commit c3fc181 into haskell:master Jan 1, 2015
@purcell
Copy link
Member

purcell commented Jan 1, 2015

Great! Merged.

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.

2 participants