Jim Meyering wrote: > I've run clang on coreutils, and it spotted quite a few dead stores. > These changes remove them: > > The chmod.c change below was not completely transparent, since > the new code returns immediately after issuing the root-dev-inode > warning, whereas the old code would fall through, and thus might > have also printed --verbose diagnostics. > > Other than that, these are supposed to be "no semantic change" patches.
These look good.