[GitHub] [qpid-dispatch] astitcher edited a comment on issue #528: DISPATCH-1375 qdr_general_work_t could be shrinked to increase cache localty

[GitBox <gi...@apache.org>]

astitcher edited a comment on issue #528: DISPATCH-1375 qdr_general_work_t could be shrinked to increase cache localty
URL: https://github.com/apache/qpid-dispatch/pull/528#issuecomment-506405116
 
 
   Having said that it would be trivial (if irritating) to just add in a struct name. In fact looking at the code I'd say this might be better so that you can clearly see which type of the union you are using.
   
   Having said that as a non-standard extension many C99 compilers support anonymous structs anyway (I'm pretty sure that gcc and clang do using -std=gnu99)

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org