[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms
From: |
Frediano Ziglio |
Subject: |
Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms |
Date: |
Wed, 23 Oct 2019 10:15:26 -0400 (EDT) |
>
> On 10/23/19 8:42 AM, Laurent Vivier wrote:
> > Le 23/10/2019 à 14:26, Frediano Ziglio a écrit :
> >> Signed-off-by: Frediano Ziglio <address@hidden>
> >> ---
> >> util/qemu-timer.c | 6 +-----
> >> 1 file changed, 1 insertion(+), 5 deletions(-)
> >>
> >> diff --git a/util/qemu-timer.c b/util/qemu-timer.c
> >> index d428fec567..094a20a05a 100644
> >> --- a/util/qemu-timer.c
> >> +++ b/util/qemu-timer.c
> >> @@ -322,11 +322,7 @@ int qemu_timeout_ns_to_ms(int64_t ns)
> >> ms = DIV_ROUND_UP(ns, SCALE_MS);
> >>
> >> /* To avoid overflow problems, limit this to 2^31, i.e. approx 25
> >> days */
> >> - if (ms > (int64_t) INT32_MAX) {
> >> - ms = INT32_MAX;
> >> - }
> >> -
> >> - return (int) ms;
> >> + return (int) MIN(ms, (int64_t) INT32_MAX);
> >> }
>
> Why so many casts? It should also work as:
>
> return MIN(ms, INT32_MAX);
>
This was former version. Laurent pointed out that MIN macro
is using ternary operator which is expected to find the same time
on second and third part so the cast inside the MIN macro.
The cast before MIN was kept from previous code.
Frediano
- Re: [PATCH 1/3] util/async: avoid useless cast, Laurent Vivier, 2019/10/21
- [PATCH 1/3] util/async: avoid useless cast, Frediano Ziglio, 2019/10/23
- [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Frediano Ziglio, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Laurent Vivier, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Eric Blake, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms,
Frediano Ziglio <=
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Eric Blake, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Laurent Vivier, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Eric Blake, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Laurent Vivier, 2019/10/23
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Laurent Vivier, 2019/10/24
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Eric Blake, 2019/10/24
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Laurent Vivier, 2019/10/24
- Re: [PATCH 3/3] qemu-timer: reuse MIN macro in qemu_timeout_ns_to_ms, Eric Blake, 2019/10/24
[PATCH 2/3] event_notifier: avoid dandling file descriptor in event_notifier_cleanup, Frediano Ziglio, 2019/10/23