/hg/icedtea6: Don't generate a random pointer from a pthread_t i...

[andrew at icedtea.classpath.org]

changeset 1e95c787d40b in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=1e95c787d40b
author: Andrew John Hughes <ahughes at redhat.com>
date: Tue Jun 15 18:34:45 2010 +0100

	Don't generate a random pointer from a pthread_t in the debug
	output.

	2010-06-14 Andrew John Hughes <ahughes at redhat.com>

	 Don't print out the return value of pthread_self
	which is a pthread_t. pthread_t is an opaque type and we
	don't know what it actually is (it varies from system to
	system; recent versions of Linux use an unsigned long int).
		* plugin/icedteanp/IcedTeaPluginUtils.h:
	(PLUGIN_DEBUG_0ARG(str)): Don't print the thread_t.
	(PLUGIN_DEBUG_1ARG(str,arg1)): Likewise.
	(PLUGIN_DEBUG_2ARG(str,arg1,arg2)): Likewise.
	(PLUGIN_DEBUG_3ARG(str,arg1,arg2,arg3)): Likewise.
	(PLUGIN_DEBUG_4ARG(str,arg1,arg2,arg3,arg4)): Likewise.
	(PLUGIN_DEBUG_5ARG(str,arg1,arg2,arg3,arg4,arg5)): Likewise.


diffstat:

2 files changed, 21 insertions(+), 6 deletions(-)
ChangeLog                             |   15 +++++++++++++++
plugin/icedteanp/IcedTeaPluginUtils.h |   12 ++++++------

diffs (79 lines):

diff -r 9a9ed35b3f80 -r 1e95c787d40b ChangeLog
--- a/ChangeLog	Mon Jun 14 15:03:55 2010 -0400
+++ b/ChangeLog	Tue Jun 15 18:34:45 2010 +0100
@@ -1,3 +1,18 @@ 2010-06-14  Deepak Bhole <dbhole at redhat.
+2010-06-14 Andrew John Hughes  <ahughes at redhat.com>
+
+	Don't print out the return value of pthread_self
+	which is a pthread_t.  pthread_t is an opaque type
+	and we don't know what it actually is (it varies
+	from system to system; recent versions of Linux
+	use an unsigned long int).
+	* plugin/icedteanp/IcedTeaPluginUtils.h:
+	(PLUGIN_DEBUG_0ARG(str)): Don't print the thread_t.
+	(PLUGIN_DEBUG_1ARG(str,arg1)): Likewise.
+	(PLUGIN_DEBUG_2ARG(str,arg1,arg2)): Likewise.
+	(PLUGIN_DEBUG_3ARG(str,arg1,arg2,arg3)): Likewise.
+	(PLUGIN_DEBUG_4ARG(str,arg1,arg2,arg3,arg4)): Likewise.
+	(PLUGIN_DEBUG_5ARG(str,arg1,arg2,arg3,arg4,arg5)): Likewise.
+
 2010-06-14  Deepak Bhole <dbhole at redhat.com>
 
 	* plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java: Fix bug
diff -r 9a9ed35b3f80 -r 1e95c787d40b plugin/icedteanp/IcedTeaPluginUtils.h
--- a/plugin/icedteanp/IcedTeaPluginUtils.h	Mon Jun 14 15:03:55 2010 -0400
+++ b/plugin/icedteanp/IcedTeaPluginUtils.h	Tue Jun 15 18:34:45 2010 +0100
@@ -71,7 +71,7 @@ exception statement from your version. *
   {                                         \
     if (plugin_debug)                       \
     {                                       \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str);                \
     }                                       \
   } while (0)
@@ -81,7 +81,7 @@ exception statement from your version. *
   {                                         \
     if (plugin_debug)                       \
     {                                       \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1);          \
     }                                       \
   } while (0)
@@ -91,7 +91,7 @@ exception statement from your version. *
   {                                         \
     if (plugin_debug)                       \
     {                                       \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2);    \
     }                                       \
   } while (0)
@@ -101,7 +101,7 @@ exception statement from your version. *
   {                                            \
     if (plugin_debug)                          \
     {                                          \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2, arg3); \
     }                                          \
   } while (0)
@@ -111,7 +111,7 @@ exception statement from your version. *
   {                                                  \
     if (plugin_debug)                                \
     {                                                \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2, arg3, arg4); \
     }                                                \
   } while (0)
@@ -121,7 +121,7 @@ exception statement from your version. *
   {                                                  \
     if (plugin_debug)                                \
     {                                                \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2, arg3, arg4, arg5); \
     }                                                \
   } while (0)