diff options
author | Yusuke Endoh <mame@ruby-lang.org> | 2019-10-09 23:02:22 +0900 |
---|---|---|
committer | Yusuke Endoh <mame@ruby-lang.org> | 2019-10-09 23:02:22 +0900 |
commit | dd477df411691803fc5a83c7daa64faac112a0e4 (patch) | |
tree | 0a95c5ef1206fee0e4340ffe07f932034943c298 | |
parent | b9cf58d2b2deca7d5b204080c2fe04d68cc50ab1 (diff) | |
download | ruby-dd477df411691803fc5a83c7daa64faac112a0e4.tar.gz |
error.c (rb_bug_for_fatal_signal): renamed from rb_bug_context
Just refactoring.
The name "rb_bug_context" is completely unclear for me.
(Can you see that "context" means "machine register context"?)
The context is available only when a fatal signal (sigbus, sigsegv, or
sigill) is received; in fact, the function is used only for fatal
signals. So, I think the name should be changed.
-rw-r--r-- | error.c | 2 | ||||
-rw-r--r-- | signal.c | 6 | ||||
-rw-r--r-- | vm_core.h | 2 |
3 files changed, 5 insertions, 5 deletions
@@ -599,7 +599,7 @@ rb_bug(const char *fmt, ...) } void -rb_bug_context(const void *ctx, const char *fmt, ...) +rb_bug_for_fatal_signal(const void *ctx, const char *fmt, ...) { const char *file = NULL; int line = 0; @@ -933,7 +933,7 @@ sigbus(int sig SIGINFO_ARG) #if defined __APPLE__ || defined __linux__ CHECK_STACK_OVERFLOW(); #endif - rb_bug_context(SIGINFO_CTX, "Bus Error" MESSAGE_FAULT_ADDRESS); + rb_bug_for_fatal_signal(SIGINFO_CTX, "Bus Error" MESSAGE_FAULT_ADDRESS); } #endif @@ -946,7 +946,7 @@ sigsegv(int sig SIGINFO_ARG) { check_reserved_signal("SEGV"); CHECK_STACK_OVERFLOW(); - rb_bug_context(SIGINFO_CTX, "Segmentation fault" MESSAGE_FAULT_ADDRESS); + rb_bug_for_fatal_signal(SIGINFO_CTX, "Segmentation fault" MESSAGE_FAULT_ADDRESS); } #endif @@ -961,7 +961,7 @@ sigill(int sig SIGINFO_ARG) #if defined __APPLE__ CHECK_STACK_OVERFLOW(); #endif - rb_bug_context(SIGINFO_CTX, "Illegal instruction" MESSAGE_FAULT_ADDRESS); + rb_bug_for_fatal_signal(SIGINFO_CTX, "Illegal instruction" MESSAGE_FAULT_ADDRESS); } #endif @@ -1624,7 +1624,7 @@ extern void rb_vmdebug_debug_print_post(const rb_execution_context_t *ec, const #define SDR() rb_vmdebug_stack_dump_raw(GET_EC(), GET_EC()->cfp) #define SDR2(cfp) rb_vmdebug_stack_dump_raw(GET_EC(), (cfp)) void rb_vm_bugreport(const void *); -NORETURN(void rb_bug_context(const void *, const char *fmt, ...)); +NORETURN(void rb_bug_for_fatal_signal(const void *, const char *fmt, ...)); /* functions about thread/vm execution */ RUBY_SYMBOL_EXPORT_BEGIN |