search

Malabarba / aggressive-indent-mode

Emacs minor mode that keeps your code always indented. More reliable than electric-indent-mode.
850 stars 36 forks source link

replace control characters with call to `kbd' #65

Closed spwhitton closed 8 years ago

spwhitton commented 8 years ago

Control characters embedded in the elisp file confuses tools like file(1) (and thus Debian packaging tools).

Pre-patch:

% file aggressive-indent.el
aggressive-indent.el: data

Post-patch:

% file aggressive-indent.el
aggressive-indent.el: Lisp/Scheme program, ASCII text

Closes #64.

Malabarba commented 8 years ago

Thanks for this. I'll be glad to merge it, but I don't think it currently works as intended. That kbd form is not being evaluated. You need to replace the quote before it with a back tick, and add a comma immediately before the (kbd...).

spwhitton commented 8 years ago

Whoops, I didn't notice the quote before the form. Thanks!

Malabarba commented 8 years ago

Thanks. The behavior is correct but now it looks like the indentation on the following lines got messed up.

spwhitton commented 8 years ago

Apologies, I was relying on aggressive-indent-mode to catch it ;)

Malabarba commented 8 years ago

Great , thanks again!