Drop unnecessary casts in getElfType()

Also declare the function static and warn if return value not used.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/patchelf.cc b/src/patchelf.cc
index 8240361..4c94175 100644
--- a/src/patchelf.cc
+++ b/src/patchelf.cc
@@ -212,10 +212,10 @@ struct ElfType
 };
 
 
-ElfType getElfType(const FileContents & fileContents)
+[[nodiscard]] static ElfType getElfType(const FileContents & fileContents)
 {
     /* Check the ELF header for basic validity. */
-    if (fileContents->size() < static_cast<off_t>(sizeof(Elf32_Ehdr)))
+    if (fileContents->size() < sizeof(Elf32_Ehdr))
         error("missing ELF header");
 
     auto contents = fileContents->data();