eddelbuettel / rcppsimdjson

Rcpp Bindings for the 'simdjson' Header Library
116 stars 13 forks source link

Integer handling #16

Closed knapply closed 4 years ago

knapply commented 4 years ago

@eddelbuettel @dcooley - Let's try this again. Addressing #13

If/when all seems well, I'll update ChangeLog, NEWS before merging.

eddelbuettel commented 4 years ago

And as you said: you may as well add ChangeLog and NEWS entry. As maintainer I sometimes tweak these a little so can I also add them -- but a first pass (which may even be final...) would be appreciated.

knapply commented 4 years ago

Feel free to tweak away!

eddelbuettel commented 4 years ago

We could have done the tweaking before the merge but six of one, half a dozen of the other.

But much smoother ride this time -- I'll just make a quick cleanup.