From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.90_1) id 1omsrz-0000S2-Ni for mharc-qemu-trivial@gnu.org; Mon, 24 Oct 2022 04:31:41 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1omsrv-0000Rl-JB for qemu-trivial@nongnu.org; Mon, 24 Oct 2022 04:31:38 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1omsrh-000272-1X for qemu-trivial@nongnu.org; Mon, 24 Oct 2022 04:31:34 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1666600277; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=Ssjlo7Hv9qY+rirSyE1bh3tnUe9WNGvrf2FQbd4xqNg=; b=idUwOkdTKOmh3Y4u34MZn3dFiLZBv+KKbr5CXwdHHbBxV7pXSO2JlS2HkjqzhXCYc92eqq MVrplcun6nShyvyYRnAUW1jM35x9rgMw8AGUql0vnp9ulObzCMFx2S3NvdhKFo71KGVqsl 6HSUTD2GniHCJZhMVxmkXus8yECkA8g= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-395-TotY8LXvNmutu2uNBqXZDw-1; Mon, 24 Oct 2022 04:31:15 -0400 X-MC-Unique: TotY8LXvNmutu2uNBqXZDw-1 Received: by mail-wm1-f71.google.com with SMTP id r18-20020a05600c35d200b003cb2ba79692so1681021wmq.5 for ; Mon, 24 Oct 2022 01:31:15 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Ssjlo7Hv9qY+rirSyE1bh3tnUe9WNGvrf2FQbd4xqNg=; b=Tzo6m7HAK+1COK79gmX/kgzte1R1+8FyLBk9bDFYbVfzhpwvyzfluouXFl9qSPNAIZ MNC8o5YjV005Tm04ysgYbGTwCFlI8HuxKQW9x0MC1mFWQS1Ny3YocMB89GLjwa9OpbYW 6LeQjS3oA6G9jTW7WiBf/QEYsvu+Doj6j/Yen65i+h0W+rfcFL2AzvWXS3yvStbfT62K wybVgoR4LbnALCdqamERCQOnXZfcZYjlPFAIH3n4sa7EFEhY4YLcwyfN4E0a8j0grnQm mHm6NCqgWsP7+NvO5rwNlMdwfnx4WhyqmTC3gTlkydMulXd2gqNxQgQa3kMCBAmMNMLM HxaQ== X-Gm-Message-State: ACrzQf2XZpE81npG6MAwJCsTZDGbuAYO3Bre/q6F3/X2QT2fNpTSK9v8 SW9SSYHg10ss6pEQv5JQAbUqBoCGsmupnRRDVj3Rn9HhXCFsKL9Rc+XHRq8mtGu4MQ9Cm3WFPHC tCnjJyRSRoPM20TEPNQ== X-Received: by 2002:a5d:5a9d:0:b0:225:2783:d6f1 with SMTP id bp29-20020a5d5a9d000000b002252783d6f1mr21269543wrb.385.1666600274505; Mon, 24 Oct 2022 01:31:14 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4S301PLJM/qiU32Yr1owJEpJWbJyDPX/SIiSFxQmR6i3HsppKDYWKvQtOZ1ouF2ixQvjyi/g== X-Received: by 2002:a5d:5a9d:0:b0:225:2783:d6f1 with SMTP id bp29-20020a5d5a9d000000b002252783d6f1mr21269520wrb.385.1666600274204; Mon, 24 Oct 2022 01:31:14 -0700 (PDT) Received: from redhat.com ([2.54.160.13]) by smtp.gmail.com with ESMTPSA id y6-20020adff146000000b00228d67db06esm9208578wro.21.2022.10.24.01.31.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Oct 2022 01:31:13 -0700 (PDT) Date: Mon, 24 Oct 2022 04:31:09 -0400 From: "Michael S. Tsirkin" To: Bin Meng Cc: qemu-devel@nongnu.org, qemu-trivial@nongnu.org, Alistair Francis , Bin Meng , Christian Schoenebeck , Gerd Hoffmann , Greg Kurz , Jason Wang , Magnus Damm , Palmer Dabbelt , Pavel Pisa , Peter Maydell , Vikram Garhwal , Yoshinori Sato , qemu-arm@nongnu.org, qemu-riscv@nongnu.org Subject: Re: [PATCH] treewide: Remove the unnecessary space before semicolon Message-ID: <20221024043057-mutt-send-email-mst@kernel.org> References: <20221024072802.457832-1-bmeng@tinylab.org> MIME-Version: 1.0 In-Reply-To: <20221024072802.457832-1-bmeng@tinylab.org> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -25 X-Spam_score: -2.6 X-Spam_bar: -- X-Spam_report: (-2.6 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.517, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001, T_SPF_TEMPERROR=0.01 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-trivial@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 24 Oct 2022 08:31:38 -0000 On Mon, Oct 24, 2022 at 03:28:02PM +0800, Bin Meng wrote: > %s/return ;/return; > > Signed-off-by: Bin Meng ack for virtio trivial tree pls > --- > > include/hw/elf_ops.h | 2 +- > hw/9pfs/9p.c | 2 +- > hw/dma/pl330.c | 2 +- > hw/net/can/can_sja1000.c | 2 +- > hw/timer/renesas_cmt.c | 2 +- > hw/timer/renesas_tmr.c | 8 ++++---- > hw/virtio/virtio-pci.c | 2 +- > target/riscv/vector_helper.c | 2 +- > target/rx/op_helper.c | 4 ++-- > ui/vnc-jobs.c | 2 +- > ui/vnc.c | 2 +- > 11 files changed, 15 insertions(+), 15 deletions(-) > > diff --git a/include/hw/elf_ops.h b/include/hw/elf_ops.h > index 7c3b1d0f6c..fbe0b1e956 100644 > --- a/include/hw/elf_ops.h > +++ b/include/hw/elf_ops.h > @@ -117,7 +117,7 @@ static void glue(load_symbols, SZ)(struct elfhdr *ehdr, int fd, int must_swab, > shdr_table = load_at(fd, ehdr->e_shoff, > sizeof(struct elf_shdr) * ehdr->e_shnum); > if (!shdr_table) { > - return ; > + return; > } > > if (must_swab) { > diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c > index aebadeaa03..76c591a01b 100644 > --- a/hw/9pfs/9p.c > +++ b/hw/9pfs/9p.c > @@ -1786,7 +1786,7 @@ static void coroutine_fn v9fs_walk(void *opaque) > err = pdu_unmarshal(pdu, offset, "ddw", &fid, &newfid, &nwnames); > if (err < 0) { > pdu_complete(pdu, err); > - return ; > + return; > } > offset += err; > > diff --git a/hw/dma/pl330.c b/hw/dma/pl330.c > index 08e5938ec7..e5d521c329 100644 > --- a/hw/dma/pl330.c > +++ b/hw/dma/pl330.c > @@ -1328,7 +1328,7 @@ static void pl330_debug_exec(PL330State *s) > } > if (!insn) { > pl330_fault(ch, PL330_FAULT_UNDEF_INSTR | PL330_FAULT_DBG_INSTR); > - return ; > + return; > } > ch->stall = 0; > insn->exec(ch, opcode, args, insn->size - 1); > diff --git a/hw/net/can/can_sja1000.c b/hw/net/can/can_sja1000.c > index e0f76d3eb3..73201f9139 100644 > --- a/hw/net/can/can_sja1000.c > +++ b/hw/net/can/can_sja1000.c > @@ -431,7 +431,7 @@ void can_sja_mem_write(CanSJA1000State *s, hwaddr addr, uint64_t val, > (unsigned long long)val, (unsigned int)addr); > > if (addr > CAN_SJA_MEM_SIZE) { > - return ; > + return; > } > > if (s->clock & 0x80) { /* PeliCAN Mode */ > diff --git a/hw/timer/renesas_cmt.c b/hw/timer/renesas_cmt.c > index 2e0fd21a36..69eabc678a 100644 > --- a/hw/timer/renesas_cmt.c > +++ b/hw/timer/renesas_cmt.c > @@ -57,7 +57,7 @@ static void update_events(RCMTState *cmt, int ch) > > if ((cmt->cmstr & (1 << ch)) == 0) { > /* count disable, so not happened next event. */ > - return ; > + return; > } > next_time = cmt->cmcor[ch] - cmt->cmcnt[ch]; > next_time *= NANOSECONDS_PER_SECOND; > diff --git a/hw/timer/renesas_tmr.c b/hw/timer/renesas_tmr.c > index d96002e1ee..c15f654738 100644 > --- a/hw/timer/renesas_tmr.c > +++ b/hw/timer/renesas_tmr.c > @@ -67,18 +67,18 @@ static void update_events(RTMRState *tmr, int ch) > int i, event; > > if (tmr->tccr[ch] == 0) { > - return ; > + return; > } > if (FIELD_EX8(tmr->tccr[ch], TCCR, CSS) == 0) { > /* external clock mode */ > /* event not happened */ > - return ; > + return; > } > if (FIELD_EX8(tmr->tccr[0], TCCR, CSS) == CSS_CASCADING) { > /* cascading mode */ > if (ch == 1) { > tmr->next[ch] = none; > - return ; > + return; > } > diff[cmia] = concat_reg(tmr->tcora) - concat_reg(tmr->tcnt); > diff[cmib] = concat_reg(tmr->tcorb) - concat_reg(tmr->tcnt); > @@ -384,7 +384,7 @@ static void timer_events(RTMRState *tmr, int ch) > tmr->tcorb[ch]) & 0xff; > } else { > if (ch == 1) { > - return ; > + return; > } > tcnt = issue_event(tmr, ch, 16, > concat_reg(tmr->tcnt), > diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c > index e7d80242b7..34db51e241 100644 > --- a/hw/virtio/virtio-pci.c > +++ b/hw/virtio/virtio-pci.c > @@ -1675,7 +1675,7 @@ static void virtio_pci_device_plugged(DeviceState *d, Error **errp) > if (virtio_host_has_feature(vdev, VIRTIO_F_IOMMU_PLATFORM)) { > error_setg(errp, "VIRTIO_F_IOMMU_PLATFORM was supported by" > " neither legacy nor transitional device"); > - return ; > + return; > } > /* > * Legacy and transitional devices use specific subsystem IDs. > diff --git a/target/riscv/vector_helper.c b/target/riscv/vector_helper.c > index b94f809eb3..0020b9a95d 100644 > --- a/target/riscv/vector_helper.c > +++ b/target/riscv/vector_helper.c > @@ -211,7 +211,7 @@ static void vext_set_elems_1s(void *base, uint32_t is_agnostic, uint32_t cnt, > return; > } > if (tot - cnt == 0) { > - return ; > + return; > } > memset(base + cnt, -1, tot - cnt); > } > diff --git a/target/rx/op_helper.c b/target/rx/op_helper.c > index 9ca32dcc82..acce650185 100644 > --- a/target/rx/op_helper.c > +++ b/target/rx/op_helper.c > @@ -286,7 +286,7 @@ void helper_suntil(CPURXState *env, uint32_t sz) > uint32_t tmp; > tcg_debug_assert(sz < 3); > if (env->regs[3] == 0) { > - return ; > + return; > } > do { > tmp = cpu_ldufn[sz](env, env->regs[1], GETPC()); > @@ -305,7 +305,7 @@ void helper_swhile(CPURXState *env, uint32_t sz) > uint32_t tmp; > tcg_debug_assert(sz < 3); > if (env->regs[3] == 0) { > - return ; > + return; > } > do { > tmp = cpu_ldufn[sz](env, env->regs[1], GETPC()); > diff --git a/ui/vnc-jobs.c b/ui/vnc-jobs.c > index 4562bf8928..886f9bf611 100644 > --- a/ui/vnc-jobs.c > +++ b/ui/vnc-jobs.c > @@ -373,7 +373,7 @@ void vnc_start_worker_thread(void) > VncJobQueue *q; > > if (vnc_worker_thread_running()) > - return ; > + return; > > q = vnc_queue_init(); > qemu_thread_create(&q->thread, "vnc_worker", vnc_worker_thread, q, > diff --git a/ui/vnc.c b/ui/vnc.c > index acb3629cd8..88f55cbf3c 100644 > --- a/ui/vnc.c > +++ b/ui/vnc.c > @@ -3085,7 +3085,7 @@ static void vnc_rect_updated(VncDisplay *vd, int x, int y, struct timeval * tv) > > rect = vnc_stat_rect(vd, x, y); > if (rect->updated) { > - return ; > + return; > } > rect->times[rect->idx] = *tv; > rect->idx = (rect->idx + 1) % ARRAY_SIZE(rect->times); > -- > 2.25.1