Re: [PATCH nf-next] lib/ts_bm: add helper to reduce indentation and improve readability

[Jeremy Sowden <jeremy@azazel.net>]

[-- Attachment #1: Type: text/plain, Size: 2041 bytes --]

On 2023-06-20, at 18:42:43 +0800, kernel test robot wrote:
> kernel test robot noticed the following build warnings:
> 
> [auto build test WARNING on linus/master]
> [also build test WARNING on v6.4-rc7 next-20230620]
> [cannot apply to nf-next/master]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
> 
> url:    https://github.com/intel-lab-lkp/linux/commits/Jeremy-Sowden/lib-ts_bm-add-helper-to-reduce-indentation-and-improve-readability/20230620-031043
> base:   linus/master
> patch link:    https://lore.kernel.org/r/20230619190803.1906012-1-jeremy%40azazel.net
> patch subject: [PATCH nf-next] lib/ts_bm: add helper to reduce indentation and improve readability
> config: x86_64-rhel-8.3-rust (https://download.01.org/0day-ci/archive/20230620/202306201819.BoQ5IT7Y-lkp@intel.com/config)
> compiler: clang version 15.0.7 (https://github.com/llvm/llvm-project.git 8dfdcc7b7bf66834a761bd8de445840ef68e4d1a)
> reproduce: (https://download.01.org/0day-ci/archive/20230620/202306201819.BoQ5IT7Y-lkp@intel.com/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202306201819.BoQ5IT7Y-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):
> 
> >> lib/ts_bm.c:101:34: warning: variable 'i' is uninitialized when used here [-Wuninitialized]
>                            bs = bm->bad_shift[text[shift-i]];
>                                                          ^
>    lib/ts_bm.c:79:16: note: initialize the variable 'i' to silence this warning
>            unsigned int i, text_len, consumed = state->offset;
>                          ^
>                           = 0
>    1 warning generated.

Whoops.  Will fix and send v2.

J.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]