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

`clojure-match-next-def` doesn't support 0 arity top-level forms. #423

Closed
emidln opened this issue Jan 26, 2017 · 1 comment
Closed

`clojure-match-next-def` doesn't support 0 arity top-level forms. #423

emidln opened this issue Jan 26, 2017 · 1 comment

Comments

@emidln
Copy link
Contributor

@emidln emidln commented Jan 26, 2017

Expected behavior

clojure-match-next-def shouldn't choke on this:

(def-shortcuts)

Actual behavior

(forward-sexp) is called [1] when the cursor position is (denoted by ^) (def-shortcuts^), and this causes the following error:

(scan-error "Containing expression ends prematurely" 9581 9582)

[1]

(forward-sexp)

Steps to reproduce the problem

Place your cursor at ^:

(def-shortcuts)^

invoke (clojure-match-next-def).

Environment & Version information

clojure-mode version information

clojure-mode (version 5.7.0-snapshot)

clojure-mode-20170120.2239

Emacs version

GNU Emacs 24.5.1 (x86_64-redhat-linux-gnu, GTK+ Version 3.18.9)
$ rpm -qi emacs
Name        : emacs
Epoch       : 1
Version     : 24.5
Release     : 10.fc23
Architecture: x86_64
... <snip> ...
Signature   : RSA/SHA256, Tue 12 Apr 2016 09:13:13 PM CDT, Key ID 32474cf834ec9c
ba
Source RPM  : emacs-24.5-10.fc23.src.rpm
Build Date  : Mon 11 Apr 2016 06:21:33 AM CDT
... <snip> ...

Operating system

Fedora 23 (Linux)

@Malabarba
Copy link
Member

@Malabarba Malabarba commented Jan 27, 2017

Thanks for catching and reporting this. Would you like to try your hand at a bugfix? It sounds like this one wouldn't be too hard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.