diff options
| author | Peter Jones <pjones@redhat.com> | 2017-10-19 13:19:34 -0400 |
|---|---|---|
| committer | Peter Jones <pmjones@gmail.com> | 2018-03-12 16:21:43 -0400 |
| commit | 0d17c4921926822cfc4284c04d4c42bda6a79515 (patch) | |
| tree | 42f21562d5777c8409d0a390e6eae612b0133a07 | |
| parent | 70a4c4a39522a4d88dda0ce17a0219b10092586a (diff) | |
| download | efi-boot-shim-0d17c4921926822cfc4284c04d4c42bda6a79515.tar.gz efi-boot-shim-0d17c4921926822cfc4284c04d4c42bda6a79515.zip | |
lib: simple_file_selector(): remove some dead code.
clang-analyzer correctly believes this:
465 int i;
466
467 i = StrLen(name) - 1;
^ Value stored to 'i' is never read
468
469 for (i = StrLen(name); i > 0; --i) {
470 if (name[i] == '\\')
471 break;
472 }
And it's right; that's completely dead code.
Signed-off-by: Peter Jones <pjones@redhat.com>
| -rw-r--r-- | lib/simple_file.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/simple_file.c b/lib/simple_file.c index 0c40bdc0..e32f78d5 100644 --- a/lib/simple_file.c +++ b/lib/simple_file.c @@ -456,8 +456,6 @@ redo: } else if (StrCmp(selected, L"../") == 0) { int i; - i = StrLen(name) - 1; - for (i = StrLen(name); i > 0; --i) { if (name[i] == '\\') break; |