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

This is a general update for the cuda mode #13

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Ergus
Copy link

@Ergus Ergus commented Apr 2, 2024

If this commit is accepted I plan to add some contributions more to the the file in the near future.

Relevant changes

  • Use a derived mode
  • Add the listed keywords to cc-mode api as extra keywords
  • Add some other popular keywords to the list (some that just came to my mind)
  • Add a function to use the list also in capf integration
  • Remove many redundant initializations
  • use explicit C++ in c-lang-const calls.
  • Update the file header to include the url

Remove redundant code
(c-other-op-syntax-tokens)
(c-primitive-type-kwds)
(c-paren-nontype-kwds)
Add missing modifiers
(c-modifier-kwds)
(c-type-modifier-prefix-kwds)
Use new functions
(cuda-mode-map) : Use defvar-keymap
Unify auto-mode-alist
(cuda-mode) : Use a define-derived-mode from c++-mode
(cuda-completion-function) : New function for capf integration
Add url and lexical-binding.
Also add a version number to track
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

1 participant