[GitHub] thrift pull request #1125: [THRIFT-3961] TConnectedClient terminates connect...

2016-11-08 Thread cmhe
GitHub user cmhe opened a pull request: https://github.com/apache/thrift/pull/1125 [THRIFT-3961] TConnectedClient terminates connection when the message… … could not be processed Signed-off-by: Claudius Heine <c...@denx.de> You can merge this pull request into

[GitHub] thrift pull request: THRIFT-2073: Fixed Thrift C++ THttpClient err...

2015-11-08 Thread cmhe
Github user cmhe closed the pull request at: https://github.com/apache/thrift/pull/543 --- 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] thrift pull request: THRIFT-3215 Replaces throw new in TJSONProt...

2015-07-08 Thread cmhe
Github user cmhe commented on the pull request: https://github.com/apache/thrift/pull/536#issuecomment-119555911 Was merged with https://github.com/apache/thrift/commit/8f11f52341f5984f77b2510094e51e6aef4c3552. --- If your project is set up for it, you can reply to this email

[GitHub] thrift pull request: THRIFT-3215 Replaces throw new in TJSONProt...

2015-07-08 Thread cmhe
Github user cmhe closed the pull request at: https://github.com/apache/thrift/pull/536 --- 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] thrift pull request: THRIFT-1522: Fixes Memory leak by wrapping ge...

2015-07-06 Thread cmhe
GitHub user cmhe opened a pull request: https://github.com/apache/thrift/pull/542 THRIFT-1522: Fixes Memory leak by wrapping getaddrinfo into a class. Sponsored-by: Roger Meier r.me...@siemens.com Signed-off-by: Claudius Heine c...@denx.de You can merge this pull request

[GitHub] thrift pull request: THRIFT-2073: Fixed Thrift C++ THttpClient err...

2015-07-06 Thread cmhe
GitHub user cmhe opened a pull request: https://github.com/apache/thrift/pull/543 THRIFT-2073: Fixed Thrift C++ THttpClient error: cannot refill buffer Fixed-by: Qiang Li liqiang...@hotmail.com Sponsored-by: Roger Meier r.me...@siemens.com Signed-off-by: Claudius Heine c

[GitHub] thrift pull request: THRIFT-1522: Fixes Memory leak by wrapping ge...

2015-07-06 Thread cmhe
Github user cmhe commented on the pull request: https://github.com/apache/thrift/pull/542#issuecomment-118762338 I have run make cross and make check without any unexpected errors. Feedback is welcome. --- If your project is set up for it, you can reply to this email and have

[GitHub] thrift pull request: THRIFT-1522: Fixes Memory leak by wrapping ge...

2015-07-06 Thread cmhe
Github user cmhe commented on the pull request: https://github.com/apache/thrift/pull/542#issuecomment-118833111 Maybe this can also be used in TNonblockingServer.cpp and TSocket.cpp. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] thrift pull request: THRIFT-1522: Fixes Memory leak by wrapping ge...

2015-07-06 Thread cmhe
Github user cmhe commented on the pull request: https://github.com/apache/thrift/pull/542#issuecomment-118848681 I moved TGetAddrInfoWrapper into its own C++ Module and integrated it into TNonblockingServer and TSocket: https://github.com/cmhe/thrift/commits/WrappedGetAddrInfo

[GitHub] thrift pull request: THRIFT-3215 Replaces throw new in TJSONProt...

2015-07-01 Thread cmhe
GitHub user cmhe opened a pull request: https://github.com/apache/thrift/pull/536 THRIFT-3215 Replaces throw new in TJSONProtocol with a normal throw Sponsored-by: Roger Meier r.me...@siemens.com Signed-off-by: Claudius Heine c...@denx.de You can merge this pull request

[GitHub] thrift pull request: THRIFT-406 Convert C++ Test to Boost::Test

2015-06-29 Thread cmhe
Github user cmhe closed the pull request at: https://github.com/apache/thrift/pull/528 --- 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] thrift pull request: THRIFT-406 Convert C++ Test to Boost::Test

2015-06-29 Thread cmhe
Github user cmhe commented on the pull request: https://github.com/apache/thrift/pull/528#issuecomment-116611833 Merged in commit: https://github.com/apache/thrift/commit/5ef662b9046d18aee224018061413ad7ee5d80a5 --- If your project is set up for it, you can reply to this email

[GitHub] thrift pull request: THRIFT-406 Convert C++ Test to Boost::Test

2015-06-25 Thread cmhe
GitHub user cmhe opened a pull request: https://github.com/apache/thrift/pull/528 THRIFT-406 Convert C++ Test to Boost::Test This converts all C++ test to Boost::Test, integrates the DenseProtocolTest as a test like the others and removes C asserts from the TMemoryBufferTest