Pádraig Brady wrote: > 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.
Thanks for looking!
