Fix #26

[chainsawriot]

No description provided.

[schochastics]

thanks!

[chainsawriot]

@schochastics More tests will be added later for the PR #3.

Suggestion:

1. Do you want to consolidate the documentation of all ada_has_* in one documentation.
2. I found that utf8::as_utf8 is probably not needed on modern enough R. It can cut the running time by ~10%. But you would at least need to set the R requirement to 4.2 (for those using Windows).

[schochastics]

@chainsawriot

1. yes. If you fix the NULL thing, do you mind doing that in the same PR?
2. Will think about that but I think it is a good idea. I also would like to get rid of the vapply for the length which is a runtime killer

[chainsawriot]

@schochastics Good, will do 1.

2. The length thing is partially fixed by calculating the char* length in C++.

[schochastics]

@chainsawriot yes I realized that too just now.