paleolimbot commented on PR #719: URL: https://github.com/apache/arrow-nanoarrow/pull/719#issuecomment-2726194191
> I have some thoughts about target naming and such but I need to take some more time and think about it and look some things up. Let me know here or elsewhere when you have time to circle back! > One thing is that I see a bunch of repeating stuff that should probably be refactored in a function, but that's true in general (see L436 onwards) so that could be done as a follow up. Great point! I've opened an issue to track this one: https://github.com/apache/arrow-nanoarrow/issues/726 . -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
