More callback refactoring #713

Merged
lhchavez merged 2 commits from more-callback-refactoring into master 2020-12-10 09:19:41 -06:00

2 Commits

Author SHA1 Message Date
lhchavez f4a35f543c More callbacks refactoring
This change:

* Gets rid of the `.toC()` functions for Options objects, since they
  were redundant with the `populateXxxOptions()`.
* Adds support for `errorTarget` to the `RemoteOptions`, since they are
  used in the same stack for some functions (like `Fetch()`). Now for
  those cases, the error returned by the callback will be preserved
  as-is.
2020-12-09 19:38:34 -08:00
lhchavez 8913f284d9 Ensure that no pointer handles leak during the test
This change makes sure that pointer handles are correctly cleaned up
during tests.
2020-12-09 19:38:33 -08:00