Make defnp support multi-arity functions

[maurolopes]

I have implemented support for multi-arity functions in the defnp macro, which previously resulted in a compile error.
When shown in the profiling report, the name of the function gets a suffix "_x", where x is the arity. This happens only in the multi-arity case. For example:

(defnp f
    ([x] (reduce + (range x)))
    ([] (f 123)))

creates two lines in the report, namely f_1 and f_0.

[ptaoussanis]

Hi Mauro, that looks great - thanks a lot!

Is this code borrowed from somewhere? Any idea how robust it is? Will try take a look in detail in the next few days then get back to you on merging.

Cheers! :-)

[maurolopes]

Hi Peter,

It is not borrowed. I wrote it myself based on your own defnp
implementation :)

I used that function name-with-attributes to parse correctly the function
name, docstring and metadata, so this should be ok.

Also, I tested functions with one and multiple arities, with and without
prepost maps, so I think everything is working.

Note that declaring a function with an extra pair of parentheses around the
arguments and the body, as in (def f ([x](inc x))), makes my code consider
it multi-arity, reporting as f_1, but I think that is ok.

Cheers!

Mauro

On Thursday, August 7, 2014, Peter Taoussanis notifications@github.com
wrote:

Hi Mauro, that looks great - thanks a lot!

Is this code borrowed from somewhere? Any idea how robust it is? Will try
take a look in detail in the next few days then get back to you on merging.

Cheers! :-)

—
Reply to this email directly or view it on GitHub
#75 (comment).

[ptaoussanis]

This seems to work well Mauro, thanks again!

[maurolopes]

Cool, Peter.

Cheers!