[PATCH 2/4] xfs_repair: remove impossible tests in process_sf_dir2
Eric Sandeen
sandeen at sandeen.net
Tue Mar 10 16:01:26 CDT 2015
We're testing for an impossible case in here:
if (i == num_entries - 1) {
...
} else {
...
if (i == num_entries - 1)
/* can't happen! */
...
}
So, remove the impossible case.
Signed-off-by: Eric Sandeen <sandeen at redhat.com>
---
diff --git a/repair/dir2.c b/repair/dir2.c
index 25793e9..7aede6a 100644
--- a/repair/dir2.c
+++ b/repair/dir2.c
@@ -928,26 +928,12 @@ _("size of last entry overflows space left in in shortform dir %" PRIu64 ", "),
do_warn(
_("size of entry #%d overflows space left in in shortform dir %" PRIu64 "\n"),
i, ino);
- if (!no_modify) {
- if (i == num_entries - 1)
- do_warn(
- _("junking entry #%d\n"),
- i);
- else
- do_warn(
- _("junking %d entries\n"),
- num_entries - i);
- } else {
- if (i == num_entries - 1)
- do_warn(
- _("would junk entry #%d\n"),
- i);
- else
- do_warn(
- _("would junk %d entries\n"),
- num_entries - i);
- }
-
+ if (!no_modify)
+ do_warn(_("junking %d entries\n"),
+ num_entries - i);
+ else
+ do_warn(_("would junk %d entries\n"),
+ num_entries - i);
break;
}
}
More information about the xfs
mailing list