Skip to content

/ valaddin

New issue

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

Jump to bottom

Distinguish between closures and functions-in-general #18

Closed
egnha opened this issue Mar 30, 2017 · 0 comments
Closed

Distinguish between closures and functions-in-general #18

egnha opened this issue Mar 30, 2017 · 0 comments
Assignees
@egnha

Comments

@egnha
Copy link
Owner

@egnha egnha commented Mar 30, 2017

To match naive expectations, vld_function should correspond to the predicate base::is.function rather than purrr::is_function. A new checker vld_closure should take the place of vld_function. Thus:

vld_function <- localize("Not function" ~ is.function)
vld_closure  <- localize("Not closure" ~ purrr::is_function)
@egnha egnha self-assigned this Mar 31, 2017
egnha added a commit that referenced this issue Apr 5, 2017
@egnha
Distinguish closures from primitives (#18)
Loading status checks…
dbec26b
@egnha egnha closed this Apr 5, 2017
egnha added a commit that referenced this issue Aug 10, 2017
@egnha
Check for closures by type, explicitly (#18)
4eb5894 
In purrr 0.2.3, is_function() returns TRUE for primitive functions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@egnha egnha

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@egnha
You can’t perform that action at this time.