Remove unnecessary computation in reazon--take

[ethan-leba]

In the case that reazon-run was set to generate 1 value,reazon--take would erroneously generate a second value and discard it.

Hey again!

I may have found a bug -- let me know what you think! If I'm not mistaken, would you mind rolling a 0.4.1 release for this? I think this could improve the performance of my package a fair bit, as I'm using a bunch of (reazon-run 1 ...)s around my codebase.

[nickdrozd]

Great catch! I checked The Reasoned Schemer to see if the bug exists there...but no, it's just a problem with this implementation.

By the way, the email for these commits is just your local mbp.lan. Do you want your real email there? I don't care either way.

[ethan-leba]

Great catch! I checked The Reasoned Schemer to see if the bug exists there...but no, it's just a problem with this implementation.

I did too, that would've been quite the find!

By the way, the email for these commits is just your local mbp.lan. Do you want your real email there? I don't care either way.

Ah, thanks for pointing that out.

[nickdrozd]

0.4.1 is released. Let me know if there are any problems.

[ethan-leba]

It looks like the version's at 0.3 in reazon.el still: https://github.com/nickdrozd/reazon/blob/0.4.1/reazon.el#L7

I'm surprised I haven't gotten any errors specifying a min version of 0.4 actually, weird.. 🤔