Github user kdubb commented on the pull request:
https://github.com/apache/thrift/pull/790#issuecomment-172646482
I am fairly sure that #797 fixes the line breaks and semi-colons in the
proper way. So, just check it with your code to make sure.
As far as performance of ARC I will just say that you attempt to be micro
optimizing code that is about to, or just has, sent data over a network
connection. Any thought to the nanosecond of time ARC *may* (and most likely
doesn't) add is not something we should be concerned about. Finally, this code
initializes static constants so discussing performance at all is unnecessary.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
