From: Cyril Hrubis <chrubis@suse.cz>
To: Joerg Vehlow <lkml@jv-coder.de>
Cc: Joerg Vehlow <joerg.vehlow@aox-tech.de>, ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 05/12] posix/conformance/interfaces: Fix all unused variable warnings
Date: Mon, 22 Nov 2021 10:43:59 +0100 [thread overview]
Message-ID: <YZtmX7MzLCJIwEMV@yuki> (raw)
In-Reply-To: <5979aa76-ca07-2bf2-58cc-33bafe4a280b@jv-coder.de>
Hi!
> > Looking at the test the whole loop here is pointless, can we remove it
> > completely?
> I thought the same, but I have never used aio, so I have no clue, if
> there are any side effects.
As far as I can tell there is none. I guess that the original test
author was simply clueless, we used to have much more code that didn't
made any sense ten years ago and you can still occasionally find some...
> So I went for a minimal?? change approach.
Let's go for that for now then. It would be nice to fix it properly
later on.
--
Cyril Hrubis
chrubis@suse.cz
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next prev parent reply other threads:[~2021-11-22 9:43 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-19 7:45 [LTP] [Patch 00/12] Fix or suppress compiler warnings in posix/conformance/interfaces Joerg Vehlow
2021-11-19 7:45 ` [LTP] [PATCH 01/12] posix/pthread_create/15-1: Supress warning Joerg Vehlow
2021-11-19 14:34 ` Cyril Hrubis
2021-11-19 7:45 ` [LTP] [PATCH 02/12] posix/mq_(timed)send/5-1: Fix error reporting Joerg Vehlow
2021-11-19 14:44 ` Cyril Hrubis
2021-11-22 6:45 ` Joerg Vehlow
2021-11-22 9:10 ` Cyril Hrubis
2021-11-19 7:45 ` [LTP] [PATCH 03/12] posix/pthread_create/*: Remove unused variable Joerg Vehlow
2021-11-19 14:51 ` Cyril Hrubis
2021-11-19 7:45 ` [LTP] [PATCH 04/12] posix/conformance/interfaces: Fix unsued-variable for testfrmw Joerg Vehlow
2021-11-19 15:07 ` Cyril Hrubis
2021-11-22 6:48 ` Joerg Vehlow
2021-11-19 7:45 ` [LTP] [PATCH 05/12] posix/conformance/interfaces: Fix all unused variable warnings Joerg Vehlow
2021-11-19 15:23 ` Cyril Hrubis
2021-11-22 6:41 ` Joerg Vehlow
2021-11-22 9:43 ` Cyril Hrubis [this message]
2021-11-19 7:45 ` [LTP] [PATCH 06/12] posix/conformance/interfaces: Fix all unused function warnings Joerg Vehlow
2021-11-19 15:27 ` Cyril Hrubis
2021-11-19 7:45 ` [LTP] [PATCH 07/12] posix/pthread_key_create/2-1: Remove invalid part of test Joerg Vehlow
2021-11-19 15:32 ` Cyril Hrubis
2021-11-19 7:45 ` [LTP] [PATCH 08/12] posix/asctime: Fix potentioal buffer overflow Joerg Vehlow
2021-11-19 15:33 ` Cyril Hrubis
2021-11-19 7:45 ` [LTP] [PATCH 09/12] posix/conformance/interfaces/pthread_*_destroy: Suppress nonnull warning Joerg Vehlow
2021-11-19 15:35 ` Cyril Hrubis
2021-11-19 7:46 ` [LTP] [PATCH 10/12] posix/conformance/interfaces: Fix all sign-compare warnings Joerg Vehlow
2021-11-19 15:37 ` Cyril Hrubis
2021-11-19 7:46 ` [LTP] [PATCH 11/12] posix/conformance/interface/mmap/5-1: Remove invalid static Joerg Vehlow
2021-11-19 15:38 ` Cyril Hrubis
2021-11-19 7:46 ` [LTP] [PATCH 12/12] posix/interface/conformance: Fix/supress all unused-result warnings Joerg Vehlow
2021-11-19 15:43 ` Cyril Hrubis
2021-11-22 6:42 ` Joerg Vehlow
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YZtmX7MzLCJIwEMV@yuki \
--to=chrubis@suse.cz \
--cc=joerg.vehlow@aox-tech.de \
--cc=lkml@jv-coder.de \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox