Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@hindol
Copy link

@hindol hindol commented Apr 24, 2020

Bring in upstream changes.

weavejester and others added 26 commits January 17, 2019 15:49
Prevent common false positives in fuzzy def rule
…lified-specs

Account for current ns when finding indents
Since weavejester#123 it is now possible to use cljfmt without integrating it with
Leiningen. This commit adds documentation to the README on how to
achieve that.

Closes weavejester#93.
…ths-project-clj

Add :paths option to :cljfmt config for lein
Forms that start with a non-symbol were not correctly being filtered
out when checking against the indentation files.

Fixes weavejester#172.
Add more guidance to indentation rules docs
This patch addresses the slowdown introduced between 0.6.4 and 0.6.5
and fixes weavejester#181.

Prior to this patch, find-namespace and ns-form? called z/sexpr on
all traversed forms. The find-namespace logic needed only parts of
resulting sexprs.

The cost of z/sexpr was non-negligible when invoked on complex forms.
As a result, z/sexpr and nested functions dominated the CPU cost of
the indentation logic.

This patch makes z/sexpr calls more fine-granular and avoids invoking
the function on nodes irrelevant to find-namespace. This results in a
tenfold speedup.
Reduce the z/sexpr cost in find-namespace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants