Skip to content
This repository has been archived by the owner on Apr 11, 2021. It is now read-only.

In js2-mode, after removing a pair of parentheses next to a function, '(' can't be inserted again. #28

Closed
crocket opened this issue Oct 8, 2013 · 3 comments

Comments

@crocket
Copy link

crocket commented Oct 8, 2013

  1. type
function B() {

}
  1. remove ()
function B {

}
  1. Try typing '(' after 'function B', and you see 'Wrong type argument: stringp, .....' in the echo area.

image

The content of err is (args-out-of-range 54 235)

@joaotavora
Copy link
Owner

Merging this into #22 as it seems to be closely related.

@crocket
Copy link
Author

crocket commented Oct 10, 2013

The commit, 7319b04, seems to fix this issue at least superficially.

I don't know if the underlying issue was fixed for good. however, congratulation.

@joaotavora
Copy link
Owner

As I explain in #22, you might want to open a issue report with the js2-mode, redirecting them to this issue, so they may consider having they custom forward-sexp implementation raise scan-errors instead of args-out-of-range. To be clear, the error is legitimate, but its subclass should change to reflects the semantics.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants