Web lists-archives.com

Re: [PATCH] staging: comedi: remove redundant variable segpos




On 11/07/18 11:32, Colin King wrote:
From: Colin Ian King <colin.king@xxxxxxxxxxxxx>

Variable segpos is being assigned but is never used hence it is redundant
and can be removed.

Cleans up clang warning:
warning: variable 'segpos' set but not used [-Wunused-but-set-variable]

Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
---
  drivers/staging/comedi/drivers/pcl816.c | 4 ++--
  drivers/staging/comedi/drivers/pcl818.c | 4 ++--
  2 files changed, 4 insertions(+), 4 deletions(-)

Looks good, thanks!

Reviewed-by: Ian Abbott <abbotti@xxxxxxxxx>

--
-=( Ian Abbott <abbotti@xxxxxxxxx> || Web: www.mev.co.uk )=-
-=( MEV Ltd. is a company registered in England & Wales. )=-
-=( Registered number: 02862268.  Registered address:    )=-
-=( 15 West Park Road, Bramhall, STOCKPORT, SK7 3JZ, UK. )=-