pitrou commented on PR #44279:
URL: https://github.com/apache/arrow/pull/44279#issuecomment-2757128297
By the way, do we want to make this option an `enum` to allow for more
options in the future?
For example:
```c++
enum Alignment {
// "natural" or "native", not sure which one is better here
kNoAlignment, kNaturalAlignment
};
Alignment alignment_requirement = kNoAlignment;
```
and perhaps one day we want to add other alternatives:
```c++
enum Alignment {
kNoAlignment, kNaturalAlignment, k64ByteAlignment
};
```
(unfortunately C++ doesn't have rich enums like Rust has)
--
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]
