From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS63949 64.71.152.0/24 X-Spam-Status: No, score=-3.0 required=3.0 tests=AWL,BAYES_00, RCVD_IN_DNSWL_NONE,RDNS_NONE,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=no autolearn_force=no version=3.4.0 Received: from 80x24.org (unknown [64.71.152.64]) by dcvr.yhbt.net (Postfix) with ESMTP id 60CCE1F424 for ; Sat, 28 Apr 2018 05:04:38 +0000 (UTC) From: Eric Wong To: spew@80x24.org Subject: [PATCH] cont.c: cleanup needless cast Date: Sat, 28 Apr 2018 05:04:38 +0000 Message-Id: <20180428050438.16831-1-e@80x24.org> List-Id: It seems like leftover development step from r59557 ("refactoring Fiber status"). I will make fiber_status use BITFIELD macro in a future commit. * cont.c (struct rb_fiber_struct): drop const from fiber_status (fiber_status_set): remove cast --- cont.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cont.c b/cont.c index e4fe644481..5171b28fb7 100644 --- a/cont.c +++ b/cont.c @@ -156,7 +156,7 @@ struct rb_fiber_struct { rb_context_t cont; VALUE first_proc; struct rb_fiber_struct *prev; - const enum fiber_status status; + enum fiber_status status; /* If a fiber invokes "transfer", * then this fiber can't "resume" any more after that. * You shouldn't mix "transfer" and "resume". @@ -223,13 +223,13 @@ rb_ec_verify(const rb_execution_context_t *ec) #endif static void -fiber_status_set(const rb_fiber_t *fib, enum fiber_status s) +fiber_status_set(rb_fiber_t *fib, enum fiber_status s) { if (0) fprintf(stderr, "fib: %p, status: %s -> %s\n", (void *)fib, fiber_status_name(fib->status), fiber_status_name(s)); VM_ASSERT(!FIBER_TERMINATED_P(fib)); VM_ASSERT(fib->status != s); fiber_verify(fib); - *((enum fiber_status *)&fib->status) = s; + fib->status = s; } void -- EW