[GitHub] trafficserver pull request: TS-2978 Reorder member variables in Ht...

2015-07-20 Thread danobi
Github user danobi closed the pull request at: https://github.com/apache/trafficserver/pull/231 --- 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

[GitHub] trafficserver pull request: TS-2978 Reorder member variables in Ht...

2015-06-19 Thread danobi
Github user danobi commented on the pull request: https://github.com/apache/trafficserver/pull/231#issuecomment-113644090 Packed a little more. Now down to 4056B. pahole results [here](http://pastebin.com/rutZmEN7) --- If your project is set up for it, you can reply to this

[GitHub] trafficserver pull request: TS-2978 Reorder member variables in Ht...

2015-06-19 Thread danobi
Github user danobi commented on the pull request: https://github.com/apache/trafficserver/pull/231#issuecomment-113575571 pahole results for [unpacked](http://pastebin.com/WwLFSrrG) pahole results for [packed](http://pastebin.com/Cr6jtZac) Although it looks like there's ano

[GitHub] trafficserver pull request: TS-2978 Reorder member variables in Ht...

2015-06-18 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/231#issuecomment-113320883 Can you show the pahole(1) results for before and after? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver pull request: TS-2978 Reorder member variables in Ht...

2015-06-18 Thread danobi
Github user danobi commented on the pull request: https://github.com/apache/trafficserver/pull/231#issuecomment-113320662 Further testing with pahole may be done later, when I get a vm setup with a supported OS. --- If your project is set up for it, you can reply to this email and h

[GitHub] trafficserver pull request: TS-2978 Reorder member variables in Ht...

2015-06-18 Thread danobi
GitHub user danobi opened a pull request: https://github.com/apache/trafficserver/pull/231 TS-2978 Reorder member variables in HttpSM State Reduce padding by grouping non four byte aligned variables together. Packing was done by hand, and pretty much only touched bools and chars.