You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "eddelbuettel (via GitHub)" <gi...@apache.org> on 2023/05/21 18:40:25 UTC

[GitHub] [arrow-nanoarrow] eddelbuettel commented on pull request #199: Remove redundant initialiser loop

eddelbuettel commented on PR #199:
URL: https://github.com/apache/arrow-nanoarrow/pull/199#issuecomment-1556254655

   Oh, I see now.  You are correct -- _if_ the second loop does not complete, _then_ we could get bitten and the first pass avoids that  My bad.   Maybe just add a comment?  (And again the sibbling routine has the `memset()` which I saw, didn't fully grok, and then didn't put two and two togther.  So may a `memset()` call is best.  Could amend PR if you want me to.)


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org