path: root/vm_backtrace.c
diff options
authorshyouhei <shyouhei@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2016-09-16 06:15:55 (GMT)
committershyouhei <shyouhei@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2016-09-16 06:15:55 (GMT)
commit2fc5210f31ad23463d7b0a0e36bcfbeee7b41b3e (patch)
treef95643295c10e0d9a6d8bec6e8c147cdbd83be4a /vm_backtrace.c
parent921c47764677d19d42cb647fd0c32e9c7440b5dc (diff)
* internal.h (WARN_UNUSED_RESULT): moved to, to
actually check its availability rather to check GCC's version. * (WARN_UNUSED_RESULT): moved to here. * (RUBY_FUNC_ATTRIBUTE): change function declaration to return int rather than void, because it makes no sense for a warn_unused_result attributed function to return void. Funny thing however is that it also makes no sense for noreturn attributed function to return int. So there is a fundamental conflict between them. While I tested this, I confirmed both GCC 6 and Clang 3.8 prefers int over void to correctly detect necessary attributes under this setup. Maybe subject to change in future. * internal.h (UNINITIALIZED_VAR): renamed to MAYBE_UNUSED, then moved to for the same reason we move WARN_UNUSED_RESULT. * (MAYBE_UNUSED): moved to here. * internal.h (__has_attribute): deleted, because it has no use now. * string.c (rb_str_enumerate_lines): refactor macro rename. * string.c (rb_str_enumerate_bytes): ditto. * string.c (rb_str_enumerate_chars): ditto. * string.c (rb_str_enumerate_codepoints): ditto. * thread.c (do_select): ditto. * vm_backtrace.c (rb_debug_inspector_open): ditto. * vsnprintf.c (BSD_vfprintf): ditto. git-svn-id: svn+ssh:// b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'vm_backtrace.c')
1 files changed, 1 insertions, 1 deletions
diff --git a/vm_backtrace.c b/vm_backtrace.c
index cbde640..d4d4d41 100644
--- a/vm_backtrace.c
+++ b/vm_backtrace.c
@@ -1173,7 +1173,7 @@ rb_debug_inspector_open(rb_debug_inspector_func_t func, void *data)
rb_debug_inspector_t dbg_context;
rb_thread_t *th = GET_THREAD();
int state;
- volatile VALUE UNINITIALIZED_VAR(result);
+ volatile VALUE MAYBE_UNUSED(result); = th;
dbg_context.cfp =>cfp;