[GitHub] thrift pull request: THRIFT-3385 warning: format ‘%lu’ expects...

2015-11-18 Thread hcorg
Github user hcorg commented on a diff in the pull request: https://github.com/apache/thrift/pull/688#discussion_r45195544 --- Diff: test/cpp/src/TestClient.cpp --- @@ -488,7 +488,7 @@ int main(int argc, char** argv) { testClient.testBinary(bin_result, string(bin_data, 256

[GitHub] thrift pull request: THRIFT-3385 warning: format ‘%lu’ expects...

2015-11-18 Thread jeking3
Github user jeking3 commented on a diff in the pull request: https://github.com/apache/thrift/pull/688#discussion_r45195379 --- Diff: test/cpp/src/TestClient.cpp --- @@ -488,7 +488,7 @@ int main(int argc, char** argv) { testClient.testBinary(bin_result, string(bin_data, 2

[GitHub] thrift pull request: THRIFT-3385 warning: format ‘%lu’ expects...

2015-11-18 Thread henrique
Github user henrique commented on a diff in the pull request: https://github.com/apache/thrift/pull/688#discussion_r45190854 --- Diff: test/cpp/src/TestClient.cpp --- @@ -488,7 +488,7 @@ int main(int argc, char** argv) { testClient.testBinary(bin_result, string(bin_data,

[GitHub] thrift pull request: THRIFT-3385 warning: format ‘%lu’ expects...

2015-11-17 Thread hcorg
Github user hcorg commented on a diff in the pull request: https://github.com/apache/thrift/pull/688#discussion_r45079568 --- Diff: test/cpp/src/TestClient.cpp --- @@ -488,7 +488,7 @@ int main(int argc, char** argv) { testClient.testBinary(bin_result, string(bin_data, 256

[GitHub] thrift pull request: THRIFT-3385 warning: format ‘%lu’ expects...

2015-11-17 Thread jeking3
Github user jeking3 commented on a diff in the pull request: https://github.com/apache/thrift/pull/688#discussion_r45072155 --- Diff: test/cpp/src/TestClient.cpp --- @@ -488,7 +488,7 @@ int main(int argc, char** argv) { testClient.testBinary(bin_result, string(bin_data, 2

[GitHub] thrift pull request: THRIFT-3385 warning: format ‘%lu’ expects...

2015-11-12 Thread henrique
Github user henrique commented on the pull request: https://github.com/apache/thrift/pull/688#issuecomment-156056686 Does it work on VS, anyone? --- 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 hav