[PATCH] erofs-utils: fix `z_erofs_fixup_insize` defined but not used
    Hongbo Li 
    lihongbo22 at huawei.com
       
    Wed May  7 19:53:33 AEST 2025
    
    
  
On 2025/5/7 16:52, Gao Xiang wrote:
> Fixes: b08e804b1dd1 ("erofs-utils: lib: wrap up zeropadding calculation")
> Signed-off-by: Gao Xiang <hsiangkao at linux.alibaba.com>
> ---
>   lib/decompress.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/decompress.c b/lib/decompress.c
> index 3f553a8..1f9daea 100644
> --- a/lib/decompress.c
> +++ b/lib/decompress.c
> @@ -9,9 +9,9 @@
>   #include "erofs/err.h"
>   #include "erofs/print.h"
>   
> -static unsigned int z_erofs_fixup_insize(const u8 *padbuf, unsigned int padbufsize)
> +static inline u32 z_erofs_fixup_insize(const u8 *padbuf, u32 padbufsize)
>   {
How about using macro to constrain it? Like I send in [1].
[1] 
https://lore.kernel.org/all/20250422123612.261764-5-lihongbo22@huawei.com/
Thanks,
Hongbo
> -	unsigned int inputmargin;
> +	u32 inputmargin;
>   
>   	for (inputmargin = 0; inputmargin < padbufsize &&
>   	     !padbuf[inputmargin]; ++inputmargin);
    
    
More information about the Linux-erofs
mailing list