commit 474796157b373730e5c09687dbdbdff11121fdcd
author Nicolas Geoffray <ngeoffray@google.com> Mon Feb 20 09:12:50 2017 +0000
committer Nicolas Geoffray <ngeoffray@google.com> Mon Feb 20 11:57:21 2017 +0000
tree 592dc056adbdf653e73c866b250d09efd7548aae
parent 39aaa8007bdb3978d056b32ce8292d23ff330fa6

Use stderr to avoid mangled print statements.

test: 909-attach-agent
Change-Id: I1037edca0cc07b0635057ac50ae5c6557e054d78

test/909-attach-agent/attach.cc

diff --git a/test/909-attach-agent/attach.cc b/test/909-attach-agent/attach.cc
index 2b50eb8..adae844 100644
--- a/test/909-attach-agent/attach.cc
+++ b/test/909-attach-agent/attach.cc
@@ -28,7 +28,7 @@
 jint OnAttach(JavaVM* vm,
             char* options ATTRIBUTE_UNUSED,
             void* reserved ATTRIBUTE_UNUSED) {
-  printf("Attached Agent for test 909-attach-agent\n");
+  fprintf(stderr, "Attached Agent for test 909-attach-agent\n");
   fsync(1);
   jvmtiEnv* env = nullptr;
   jvmtiEnv* env2 = nullptr;
@@ -36,7 +36,7 @@
 #define CHECK_CALL_SUCCESS(c) \
   do { \
     if ((c) != JNI_OK) { \
-      printf("call " #c " did not succeed\n"); \
+      fprintf(stderr, "call " #c " did not succeed\n"); \
       return -1; \
     } \
   } while (false)
@@ -44,7 +44,7 @@
   CHECK_CALL_SUCCESS(vm->GetEnv(reinterpret_cast<void**>(&env), JVMTI_VERSION_1_0));
   CHECK_CALL_SUCCESS(vm->GetEnv(reinterpret_cast<void**>(&env2), JVMTI_VERSION_1_0));
   if (env == env2) {
-    printf("GetEnv returned same environment twice!\n");
+    fprintf(stderr, "GetEnv returned same environment twice!\n");
     return -1;
   }
   unsigned char* local_data = nullptr;
@@ -54,19 +54,19 @@
   unsigned char* get_data = nullptr;
   CHECK_CALL_SUCCESS(env->GetEnvironmentLocalStorage(reinterpret_cast<void**>(&get_data)));
   if (get_data != local_data) {
-    printf("Got different data from local storage then what was set!\n");
+    fprintf(stderr, "Got different data from local storage then what was set!\n");
     return -1;
   }
   CHECK_CALL_SUCCESS(env2->GetEnvironmentLocalStorage(reinterpret_cast<void**>(&get_data)));
   if (get_data != nullptr) {
-    printf("env2 did not have nullptr local storage.\n");
+    fprintf(stderr, "env2 did not have nullptr local storage.\n");
     return -1;
   }
   CHECK_CALL_SUCCESS(env->Deallocate(local_data));
   jint version = 0;
   CHECK_CALL_SUCCESS(env->GetVersionNumber(&version));
   if ((version & JVMTI_VERSION_1) != JVMTI_VERSION_1) {
-    printf("Unexpected version number!\n");
+    fprintf(stderr, "Unexpected version number!\n");
     return -1;
   }
   CHECK_CALL_SUCCESS(env->DisposeEnvironment());