felipecrv commented on issue #45808:
URL: https://github.com/apache/arrow/issues/45808#issuecomment-2735184228
My understand has always been `arrow::internal` can leak into headers, but
`arrow::detail` is for details within a `.cc` (compilation unit).
`arrow::internal` APIs MUST be present in headers because to implement
public APIs across different `.cc` files, we need the internal APIs.
Number of .h and .cc where these names occur:
```
~/code/arrow (main)$ ag "\bdetail\b" -l | egrep "\.cc$" | wc -l
50
~/code/arrow (main)$ ag "\bdetail\b" -l | egrep "\.h$" | wc -l
46
~/code/arrow (main)$ ag "\binternal\b" -l | egrep "\.h$" | wc -l
304
~/code/arrow (main)$ ag "\binternal\b" -l | egrep "\.cc$" | wc -l
600
```
--
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]
