Notes |
|
(0031055)
|
Eric NOULARD
|
2012-09-19 04:59
|
|
I'm currently reviewing the patch. |
|
|
(0031056)
|
Eric NOULARD
|
2012-09-19 05:12
|
|
The feature is nice but why did you change
cmake-regex-block-open/close from possibly mixed case to upper case?
I'm not an emacs mode expert so, wouldn't this prevent
correct indentation when open/close block are lower case?
I ask that because from my testing it seems to work with upper and lower
case with the new cmake-mode.el file but I don't quite understand why... |
|
|
(0031059)
|
tschutter
|
2012-09-19 09:41
|
|
I made no such change to cmake-regex-block-open/close. The version of cmake-mode.el that you are looking at must be different than the one available at http://www.cmake.org/CMakeDocs/cmake-mode.el, [^] which was my starting point.
But you have identified an actual bug. `cmake-regex-block-open' is passed to the string-match function. The string-match function ignores case if `case-fold-search' is non-nil. The default for `case-fold-search' is t. But `case-fold-search' can be modified by the user.
Hold until I can modify cmake-mode.el, test, and attach a new version. |
|
|
(0031060)
|
tschutter
|
2012-09-19 10:06
|
|
Eric, can you provide to me the original version of cmake-mode.el that you were comparing against? That way I can make sure to incorporate any changes that it has. |
|
|
(0031061)
|
Eric NOULARD
|
2012-09-19 10:34
|
|
|
|
(0031062)
|
Eric NOULARD
|
2012-09-19 10:36
|
|
If you know git you can clone the CMake git repo
http://www.cmake.org/Wiki/CMake/Git [^]
and provide a patch,
you can use
git format-patch HEAD~1
if your change is contained in a single local commit. |
|
|
(0031069)
|
tschutter
|
2012-09-19 22:53
|
|
I have attached two patches.
The first patch reimplements ab9824e using more readable regexes and the case-fold-search function.
The second patch adds the keymap. |
|
|
(0031077)
|
Eric NOULARD
|
2012-09-21 00:34
|
|
Thanks a lot.
I just committed you patch to next. I did a small edit of the commit log
message because it was reject by commit hook (no more than 78 characters
for the first comment line)
Merge topic 'UpdateEmacsMode-bug0013548' into next
301e67e cmake-mode.el: add local keybindings
6d1e4ef cmake-mode.el: Use more readable regex and case-fold-search
Should be on master on next tuesday unless someone discover something
wrong with this in the meantime.
Thank you for your contribution. |
|
|
(0032455)
|
Robert Maynard
|
2013-03-04 08:38
|
|
Closing resolved issues that have not been updated in more than 4 months. |
|