On Tue, Aug 28, 2018 at 02:21:01PM +0100, Colin King wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > Pointer 'tree' is being assigned but is never used hence it is > redundant and can be removed. For patches removing dead code it's good to do a bit of research what was the reason for that. In most cases it's a simple leftover after some other cleanup or rework, ie. harmless, but this needs to be reviewed anyway and the reference in changelog helps. In this case it's after my commit 00032d38eaa89c76de7, I'll update the changelog. And the previous patch removing inode in relink_file_extents was udpated too. > > Cleans up clang warning: > warning: variable 'tree' set but not used [-Wunused-but-set-variable] > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Reviewed-by: David Sterba <dsterba@xxxxxxxx>
