WillAyd commented on issue #599: URL: https://github.com/apache/arrow-nanoarrow/issues/599#issuecomment-2451104863
Are we definitely locked into C++17? If not, I wonder if it wouldn't be better to return a std::span instead of a data pointer. As far as error handling is concerned, maybe we could use C++ exceptions? Having used this downstream, I've always found the `NANOARROW_THROW_NOT_OK` macro to be a nice convenience, so might be nice to continue using that pattern for members that can throw -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
