lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <5388E360-C450-4F54-B4B5-00568165FBC2@gmail.com>
Date:   Tue, 28 Apr 2020 20:08:35 -0300
From:   Arnaldo Melo <arnaldo.melo@...il.com>
To:     Joe Perches <joe@...ches.com>, Zou Wei <zou_wei@...wei.com>
CC:     peterz@...radead.org, mingo@...hat.com, mark.rutland@....com,
        alexander.shishkin@...ux.intel.com, jolsa@...hat.com,
        namhyung@...nel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH -next] perf script: remove unneeded conversion to bool



On April 28, 2020 6:58:46 PM GMT-03:00, Joe Perches <joe@...ches.com> wrote:
>On Tue, 2020-04-28 at 14:50 -0300, Arnaldo Carvalho de Melo wrote:
>> Em Sun, Apr 26, 2020 at 08:31:05PM +0800, Zou Wei escreveu:
>> > This issue was detected by using the Coccinelle software:
>> > 
>> > tools/perf/builtin-script.c:3285:48-53: WARNING: conversion to bool
>not needed here
>> > tools/perf/builtin-script.c:2839:36-41: WARNING: conversion to bool
>not needed here
>> > 
>> > The conversion to bool is unneeded, remove it
>> 
>> Thanks, applied,
>
>Did you read the patch?
>It's broken, unless some other patch was applied.

Of course I read it, but yeah, I made a mistake, thanks for pointing out, I'll remove it, thanks.
-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ