-
Notifications
You must be signed in to change notification settings - Fork 26
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
longs? #35
Comments
"those"? |
oh, "longs". perhaps a bit more exposition might help clarify the question for me. |
Oop, yep. So we have a lot of functions returning say std::vector < std::long > ... or accepting those as arguments. Do we have R-side support uniformly for that? I wasn't aware base was now cool with longs as a general thing. |
As long as it isn't an unsigned long, aye.
(I did not test that everywhere) which means we may need to re-think the params/returns since IPv4 conversions are going to beed the full unsigned space. |
Mmmn yeah. Would using doubles make more sense/give us enough space? Also means we can build in the NA support. |
The codebase now includes those. Do we have that supported, R-side?
The text was updated successfully, but these errors were encountered: