romankarlstetter opened a new pull request, #46264:
URL: https://github.com/apache/arrow/pull/46264

   This fixes issue #45608 by just moving the definition of the classes 
`Ticket`, `Location` and `FlightEndpoint` before `FlightInfo`.
   
   ### Rationale for this change
   `FlightInfo` class in `types.h` uses `FlightEndpoint` in a vector, and clang 
has problems building this if the definition of `FlightEndpoint` comes after 
the definition of `FlightInfo`.
   
   ### What changes are included in this PR?
   By moving the classes a bit up in the file, the problem is resolved.
   
   ### Are these changes tested?
   Compilation (with GCC) still works, and including the results in another 
project that is compiled with clang is now successful.
   
   ### Are there any user-facing changes?
   Not really. The only change is that it should work for clang now 🥳
   


-- 
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

Reply via email to