-
Notifications
You must be signed in to change notification settings - Fork 357
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
Develop -> Master #604
Develop -> Master #604
Conversation
make sure to calculate switch_offset even when using 'c-style' and fix block comments at the very top of a file
* fix ftplugin var and clean indent
* remove line comment appended to code
* add label support
* case-sensitivity
easier to read in one line
javascript allows for much more than just [A-Za-z] for identifiers. this is also faster on more recent versions of vim.
so it appears that my testing didn't reveal this because I use (and apparently most people use): ``` function a() { var i = 1, c = 2 } ``` instead of: ``` function a(){ var i = 1, c = 2 } ```
keep forgetting this
* Readme Updates * Update README.md * Update README.md * Update README.md * Update README.md * Update README.md
this uses the same algorithm from https://github.com/sweet-js/sweet.js/wiki/design#give-lookbehind-to-the-reader which the acorn parser also uses. it actually is basically the same as what i had originally but with support for labeled block statements and plain block statements.
cleaning and improving readability
i'll probably squash this down when I merge (not going to rebase it though) since they mostly consist of iterative changes which over-wrote each other. With this update there shouldn't be any noticeable changes to the results of indenting,though the code is more readable and robust |
@amadeus I think this can be merged, i'm prepared to fix any resulting bugs if i'm wrong about the stability. Doesn't seem anyone else is testing this either Later on tonight or tomorrow? |
I want to look into the I'll try to make a point of getting the |
these don't seem to be very relevant when using indentexprs, and may override the indentation script, opinions on this are welcomed, and honestly I don't know exactly what effect these have. the clojure indent file and a few others included with vim has these options set to off
small regex improvements
have any changes for this update @amadeus ? if not, tomorrow i'll merge, otherwise i will wait if you care for making this a full repo update |
better to get this out now is my opinion. It is a moderately sized indent update, and seems stable |
All indent related. By the way, this will be making it's way into vim(as an update to the slightly outdated file from this repo already in the newest releases), so please try it and report bugs!
I know of no regressions caused by these commits.