[前][次][番号順一覧][スレッド一覧]

ruby-changes:51243

From: normal <ko1@a...>
Date: Thu, 17 May 2018 12:48:42 +0900 (JST)
Subject: [ruby-changes:51243] normal:r63450 (trunk): vm_trace: remove rb_postponed_job_t->flags

normal	2018-05-17 12:48:32 +0900 (Thu, 17 May 2018)

  New Revision: 63450

  https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=63450

  Log:
    vm_trace: remove rb_postponed_job_t->flags
    
    This field has been taking up space for 5 years and never used.
    Save 8kb of memory on x86-64 for now.  If we ever care about
    this flag, we'll simply re-add it.
    
    * vm_trace.c (rb_postponed_job_struct): remove flags
      (postponed_job_register): comment out flags assignment
      [ruby-core:87052] [Misc #14764]

  Modified files:
    trunk/vm_trace.c
Index: vm_trace.c
===================================================================
--- vm_trace.c	(revision 63449)
+++ vm_trace.c	(revision 63450)
@@ -1518,7 +1518,6 @@ Init_vm_trace(void) https://github.com/ruby/ruby/blob/trunk/vm_trace.c#L1518
 }
 
 typedef struct rb_postponed_job_struct {
-    unsigned long flags; /* reserved */
     rb_postponed_job_func_t func;
     void *data;
 } rb_postponed_job_t;
@@ -1555,7 +1554,7 @@ postponed_job_register(rb_execution_cont https://github.com/ruby/ruby/blob/trunk/vm_trace.c#L1554
 	return PJRR_INTERRUPTED;
     }
 
-    pjob->flags = flags;
+    /* unused: pjob->flags = flags; */
     pjob->func = func;
     pjob->data = data;
 

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]