We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
using Lazy @bounce pow(x, n ; res=1) = n > 0 ? pow(x, n-1; res=res*x) : res
gives the error:
syntax: invalid syntax ; res = 1 Stacktrace: [1] top-level scope at In[2]:2 [2] include_string(::Function, ::Module, ::String, ::String) at .\loading.jl:1091
because the macro definition doesn't account for this case. At https://github.com/MikeInnes/Lazy.jl/blob/master/src/tail.jl#L134, the function definition should be;
$f($(args...);$(kwargs...)) = trampoline($f_tramp, $(args...); $(kwargs...))
instead of
$f($(args...)) = trampoline($f_tramp, $(args...))
with args, kwargs defined by pattern-matching rather than directly indexing args arrays. Minor PR incoming...
The text was updated successfully, but these errors were encountered:
handle keyword arguments in bounce macro
a236aff
addresses issue MikeInnes#124
No branches or pull requests
gives the error:
because the macro definition doesn't account for this case.
At https://github.com/MikeInnes/Lazy.jl/blob/master/src/tail.jl#L134, the function definition should be;
instead of
with args, kwargs defined by pattern-matching rather than directly indexing args arrays. Minor PR incoming...
The text was updated successfully, but these errors were encountered: