Try setting it to (lambda () 0)

I'm not 100% sure if the indent spec accepts a function, but it might.

I'll try tomorrow thanks Malabarba

How does clojure-defun-indents work?

I tried setting it as a file-local variable but it doesn't seem to have any effect.

@stuartsierra: Looks like the way it's designed it won't work as a file-local variable.

It won't even work if you setq it either 🙂

It'll only work by configuring it via the customize menu

Obviously, the variable's doc shouldn't say that: > This variable is safe as a file local variable if its value satisfies the predicate ‘listp’.

ah, thanks @malabarba

@bozhidar I think this variable is left-over from a previous version of the indentation engine. We should either change the way it's used so that it works as a file-local, or make it obsolete and tell people to use define-clojure-indent instead.

@stuartsierra: You can still have file-local indent by putting something like this as a file-local eval: (define-clojure-indent (my-fn :defn) (some-def :defn))

Yes, that's what I've been doing.

Ok then 🙂

guess we should make it obsolete

define-clojure-ident is the way to go

although we should have probably named this clojure-define-indentation 🙂

mmm no, I don't think (trace-handlers (lambda () 0) works here...

the doc of clojure-indent-function is pretty clear:

The property value can be

- `defun', meaning indent `defun'-style;
- an integer N, meaning indent the first N arguments specially
  like ordinary function arguments and then indent any further
  arguments like a body;
- a function to call just as this function was called.
  If that function returns nil, that means it doesn't specify
  the indentation.
- a list, which is used by `clojure-backtracking-indent'.

so I tried to return nil, but the minimum indendantation I get is 1 space, probably taking into consideration the initial paren

(rf/trace-handlers 
 (rf/reghandler :reset-db app-mw
                [_ []]
                db/initial-state))

with (trace-handlers (lambda () nil))

Returning nil means it doesn't specify indentation

that's not the same as specifying 0 indentation

that just means "not my problem, do the default"

@richiardiandrea: Try this instead: (define-clojure-indent (trace-handlers (lambda (&rest _) 0)))

I forgot the function has to take arguments

Oh ok, I also opened an issue in clojure-mode to add :never-align

I was playing with clojure-mode earlier

It looks like that 0 is not directly the column shift

So that is why I was thinking of adding a keyword to handle that case

@malabarba: I checked the code and it looks like your option should work but for some reason (functionp method) returns nil here https://github.com/clojure-emacs/clojure-mode/blob/master/clojure-mode.el#L1238

where method is the lambda (not an emacs expert here so maybe it does not match against lambdas?)

Odd. Can you see what's the value of method in that case?

edebug returns lambda

the doc says and returns nil for special forms

again, dunno what this means in elisp though 🙂

ah @malabarba ok, lambda is a special form 😄

everything makes sense now ahah

hm

it might be a bug in our code

Yep

yeah

just found it

I opened an issue, I can take care of the pr if you tell which is the right function 😉

but I still think that a :never-align option might be good

so #'(lambda () ... should work right?

I don't think so

for the same reason

true, I tried

I've already written the fix

great, I can try it out

If you'd like to write a couple of tests I can let you open the PR 🙂

I am adding the test

@malabarba: no honey with (trace-handlers (lambda (_ _) 0))

Weird

it worked for me with (define-clojure-indent (trace-handlers (lambda (&rest _) 0)))

I'll check again

the test passes