Skip to content

Support testing/clearing Assignable_Proxy_Function's callable#480

Open
totalgee wants to merge 1 commit intoChaiScript:developfrom
totalgee:function_clear
Open

Support testing/clearing Assignable_Proxy_Function's callable#480
totalgee wants to merge 1 commit intoChaiScript:developfrom
totalgee:function_clear

Conversation

@totalgee
Copy link
Contributor

@totalgee totalgee commented Apr 21, 2019

Issue this pull request references: #321

Changes proposed in this pull request

  • add operator bool() cast to test if an Assignable_Proxy_Function has a callable.
  • add clear() to reset/empty the callable function.
  • include new compiled test case to work with (setting, testing, calling and clearing) callback functions in C++ objects.

@totalgee
Support testing/clearing Assignable_Proxy_Function's callable
8af9cc5 
- add `operator bool()` cast to test if an Assignable_Proxy_Function
  has a callable.
- add `clear()` to reset/empty the callable function.
- include new compiled test case to work with (setting, testing, calling
  and clearing) callback functions in C++ objects.
@totalgee
Copy link
Contributor Author

totalgee commented Apr 21, 2019

Incidentally, when I run make test or just ctest locally, if there is actually an error it still shows the tests as passing. However if you actually run ./compiled_tests you'll see if there were any failing exceptions or CHECK() clauses. Why is that? Does "compiled_tests" not have the proper return code? Hopefully it actually checks these things properly when running the continuous integration, if not in my local build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@totalgee