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

Emacs mode: Several additions & adjustments #146

Closed
wants to merge 4 commits into from
Closed

Emacs mode: Several additions & adjustments #146

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 6, 2015

Define cmake-mode properly with define-derived-mode
Add font-lock for keywords and variables
Rename to cmake-unscreamify-buffer
cmake-indent: Make closing parens match opening; clean up some

@ghost ghost changed the title Several additions & adjustments Several additions & adjustments to Emacs mode Mar 7, 2015
@ghost ghost changed the title Several additions & adjustments to Emacs mode Emacs mode: Several additions & adjustments Mar 7, 2015
@bradking
Copy link
Member

bradking commented Mar 9, 2015

Thanks. Please revise the commit messages to start in cmake-mode.el: as is the convention in the ancestry of that file.

This allows the keywords be defined in a list and reused in different regexps.
@ghost
Copy link
Author

ghost commented Mar 9, 2015

@bradking Done, hope that looks okay.

Roy Crihfield added 3 commits March 9, 2015 17:04
Rename `unscreamify-cmake-buffer' to `cmake-unscreamify-buffer'.
Also, enclose function body with `save-excursion'.
Use `define-derived-mode` as per the recommended method:
https://www.gnu.org/software/emacs/manual/html_node/elisp/Derived-Modes.html
This automatically handles setup of the syntax table, keymaps, etc.
@ghost
Copy link
Author

ghost commented Mar 9, 2015

I've rebased to leave out the problem commits https://github.com/roysc/CMake/commit/8f83081b3687857d9f372d31a1de543a78de4148 and https://github.com/roysc/CMake/commit/d9d7b027354f5314f658aaae707bdc67b10f0f84. I'll open a new PR for those later.

@bradking
Copy link
Member

Please look at my comment on commit 31420cf. Currently it breaks indentation. Thanks.

@ghost
Copy link
Author

ghost commented Mar 17, 2015

I saw that - I have some idea of the problem but I will have to test a bit more carefully this time. Sorry for the mess.

@bradking
Copy link
Member

bradking commented Apr 1, 2015

Closing for now. Please re-open when new revisions are ready.

@bradking bradking closed this Apr 1, 2015
@bradking
Copy link
Member

Corresponding CMake issue:

http://www.cmake.org/Bug/view.php?id=15521

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