From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on gnuweeb.org X-Spam-Level: X-Spam-Status: No, score=-1.8 required=5.0 tests=ALL_TRUSTED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,NO_DNS_FOR_FROM, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from [10.7.7.5] (unknown [182.253.183.172]) by gnuweeb.org (Postfix) with ESMTPSA id 550D2804D1; Thu, 20 Oct 2022 12:12:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1666267930; bh=sLQzqjMZzRlc+ryGIyihVk2qTT6WrL4vgWF+XKpJdlU=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=Tk/K8OwLpQ8gElAUMAAQKGXQKKC3rqVIQLIeuW8IaQQBw54uo78MdvJFWU4X06Mf1 wxsEVrHqsYzFUQoE4Ky91tHRNZj/LbcO8xsFP+EzcvkUJLXaCOtulD1YOeUgl6WFpl hjy72XN6qv9l5ykatH4j1xOgfcaOQZ8T6BfPjl1CL2JI8WH9+T2sVC7rdA+liE145T QT3yOb8EnBr8lmGZnb05Uj98lvpEGNb9cSAPqsCjUNGQQjjAugmdwTePUZlDLM4FrP uoZ0HTSRfSh03moVhd1lrs/yrBTJPHh1hZFIq9VyTJsdI/x5Z2KQnnZ81nCAbP2qNw ECF3Yj9YKGwmw== Message-ID: <0e72af49-9e6c-2cc2-c53c-3966b20517cf@gnuweeb.org> Date: Thu, 20 Oct 2022 19:12:05 +0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: [PATCH liburing v1 1/3] liburing: Clean up `-Wshorten-64-to-32` warnings from clang Content-Language: en-US To: Jens Axboe , Dylan Yudaken Cc: Pavel Begunkov , GNU/Weeb Mailing List , io-uring Mailing List , Facebook Kernel Team , Dylan Yudaken References: <20221020114814.63133-1-ammar.faizi@intel.com> <20221020114814.63133-2-ammar.faizi@intel.com> From: Ammar Faizi In-Reply-To: <20221020114814.63133-2-ammar.faizi@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: On 10/20/22 6:52 PM, Ammar Faizi wrote: > From: Dylan Yudaken > > liburing has a couple of int shortening issues found by clang. Clean > them all. This cleanup is particularly useful for build systems that > include these files and run with that error enabled. > > Link:https://lore.kernel.org/io-uring/20221019145042.446477-1-dylany@meta.com > Signed-off-by: Dylan Yudaken > Co-authored-by: Ammar Faizi > Signed-off-by: Ammar Faizi BTW, before it's too late. I think we should be consistent on the cast style: (type) a or (type)a What do you think? -- Ammar Faizi