[GitHub] thrift issue #1328: THRIFT-2221: detect C++11 and use std namespace for memo...

2017-08-10 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1328 The lone build failure is a "D" language/test issue that appears sporadically. All other builds passed. As such, I am going to merge this in given it was already reviewed, and the follow-on chang

[GitHub] thrift issue #1328: THRIFT-2221: detect C++11 and use std namespace for memo...

2017-08-10 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1328 I added a TPipe test because I changed one line of code in TPipe to work around a build issue, and I didn't see a TTransportTest test for TPipe. --- If your project is set up for it, you can reply

[GitHub] thrift issue #1328: THRIFT-2221: detect C++11 and use std namespace for memo...

2017-08-09 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1328 ![image](https://user-images.githubusercontent.com/11771245/29127276-a0e3e7da-7cee-11e7-9149-0e2c6f143f91.png) Still pushing changes here. Microsoft didn't make things easy by leaving __cp