* thread_pthread.c (consume_communication_pipe):

  Make "buff" as static.  (Maybe) "buff" can be shared between
  any caller (any threads) because no one use the read values.
  And remove const value "buff_size", and define CCP_READ_BUFF_SIZE
  macro.



git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@32266 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 5 insertions, 7 deletions

diff --git a/thread_pthread.c b/thread_pthread.c
index 5726e3ba93..e78c10fb46 100644
--- a/thread_pthread.c
+++ b/thread_pthread.c
@@ -1021,15 +1021,13 @@ rb_thread_wakeup_timer_thread(void)
 static void
 consume_communication_pipe(void)
 {
-    const size_t buff_size = 1024;
-#ifdef __FreeBSD__
-    char buff[buff_size];
-#else
-    char buff[1024];
-#endif
+#define CCP_READ_BUFF_SIZE 1024
+    /* buffer can be shared because no one refers to them. */
+    static char buff[CCP_READ_BUFF_SIZE]; 
     ssize_t result;
+
   retry:
-    result = read(timer_thread_pipe[0], buff, buff_size);
+    result = read(timer_thread_pipe[0], buff, CCP_READ_BUFF_SIZE);
     if (result < 0) {
 	switch (errno) {
 	  case EINTR: goto retry;