Messages in this thread | | | Subject | Re: [PATCH] staging: comedi: remove redundant variable segpos | From | Ian Abbott <> | Date | Wed, 11 Jul 2018 18:15:55 +0100 |
| |
On 11/07/18 11:32, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > 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@canonical.com> > --- > 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@mev.co.uk>
-- -=( Ian Abbott <abbotti@mev.co.uk> || 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. )=-
| |