remove.c: combine two helper functions, to avoid sign-extension
authorJim Meyering <meyering@redhat.com>
Wed, 1 Oct 2008 10:21:58 +0000 (12:21 +0200)
committerJim Meyering <meyering@redhat.com>
Wed, 1 Oct 2008 10:27:10 +0000 (12:27 +0200)
commit94c59c97f9bed27ac820c5501c1e3a531ea918f0
treec3154caa4ffe13841f020bb29665fdf1249d02b1
parentab02e256e957a0b0b17cc00b39415862753629bd
remove.c: combine two helper functions, to avoid sign-extension

* src/remove.c (fs_handles_readdir_ordered_dirents_efficiently):
Remove function, so as not to have to worry about the type of
statfs.f_type and sign extension.
(dirent_inode_sort_may_be_useful): Adjust comment.
Perform the switch directly on the struct.member here, instead.
Andreas Schwab spotted the potential for a sign-extension bug,
that happens not to bite for the S_* f_type values currently used.
src/remove.c