[GitHub] trafficserver pull request: TS-4365: Prevent invalid clang-analyze...

2016-04-20 Thread bgaff
Github user bgaff closed the pull request at: https://github.com/apache/trafficserver/pull/592 --- 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 i

[GitHub] trafficserver pull request: TS-4365: Prevent invalid clang-analyze...

2016-04-20 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/592#issuecomment-212725625 @zwoop already resolved in c953da0 --- 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 d

[GitHub] trafficserver pull request: TS-4365: Prevent invalid clang-analyze...

2016-04-20 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/592#issuecomment-212725520 With the exception for that one unused GlobalPlugin *plugin we discussed, +1! Thanks! --- If your project is set up for it, you can reply to this email a

[GitHub] trafficserver pull request: TS-4365: Prevent invalid clang-analyze...

2016-04-20 Thread bgaff
GitHub user bgaff opened a pull request: https://github.com/apache/trafficserver/pull/592 TS-4365: Prevent invalid clang-analyzer memory leak warnings clang-analyzer is currently detecting an invalid memory leak, this PR fixes those warnings by saving the plugin pointers. You can m