Use !empty() instead of length()

Blake-Madden · Blake-Madden · commit 5a70f0f8bfb1 · 2025-06-15T09:57:56.000-04:00
diff --git a/exif.cpp b/exif.cpp
@@ -380,7 +380,7 @@ IFEntry parseIFEntry_temp(const unsigned char *buf, const unsigned offs,
       }
       // and cut zero byte at the end, since we don't want that in the
       // std::string
-      if (result.length() &&
+      if (!result.empty() &&
           result.val_string()[result.val_string().length() - 1] == '\0') {
         result.val_string().resize(result.val_string().length() - 1);
       }

Original file line number	Diff line number	Diff line change
`@@ -380,7 +380,7 @@ IFEntry parseIFEntry_temp(const unsigned char *buf, const unsigned offs,`
`380`	`380`	`}`
`381`	`381`	`// and cut zero byte at the end, since we don't want that in the`
`382`	`382`	`// std::string`
`383`		`- if (result.length() &&`
	`383`	`+ if (!result.empty() &&`
`384`	`384`	`result.val_string()[result.val_string().length() - 1] == '\0') {`
`385`	`385`	`result.val_string().resize(result.val_string().length() - 1);`
`386`	`386`	`}`