off-by-one - Githubissues

Wilfred / bison-mode

Emacs major mode for Bison/Jison, Yacc and Lex grammars

27 stars 21 forks source link

off-by-one #3

Closed tesch1 closed 9 years ago

tesch1 commented 9 years ago

on line 450, the forward-sexp starts one character after the opening brace rather than on it... point should be moved back one prior to the forward-sexp.

Wilfred commented 9 years ago

I've pushed a fix, please let me know if that fixes the issue you're seeing. This is a part of the code that I'm very unfamiliar with.

tesch1 commented 9 years ago

looks good! sorry, shoudl have included a test case...

any production with an empty (or non-sexp) {} was causing it, ie.. in the rule below it would not indent prod2 correctly...

rule: prod {} | prod2 {} ;