[GitHub] thrift pull request: Fixes THRIFT-2817 and THRIFT-2839

2015-04-20 Thread bivas
Github user bivas commented on the pull request: https://github.com/apache/thrift/pull/273#issuecomment-94379082 @Jens-G unfortunately, no. Was pointing this out so maybe @wingedkiwi can have a quick look (and maybe a fix) on this issue. --- If your project is set up for it, you can

[GitHub] thrift pull request: Fixes THRIFT-2817 and THRIFT-2839

2015-04-19 Thread bivas
Github user bivas commented on the pull request: https://github.com/apache/thrift/pull/273#issuecomment-94269795 We're still experiencing read errors with this fix merged (`Not enough frame size 4085 to read 4096 bytes`) The commit is missing a `return` statement in ca