Apply suggestion

pitrou · pitrou · commit 7fb554ca50ee · 2024-02-20T11:14:31.000+01:00
diff --git a/cpp/src/arrow/filesystem/s3fs.cc b/cpp/src/arrow/filesystem/s3fs.cc
@@ -1221,7 +1221,7 @@ bool IsDirectory(std::string_view key, const S3Model::HeadObjectResult& result)
     return false;
   }
   // Otherwise, if it has a trailing slash, it's a directory
-  if (key[key.size() - 1] == kSep) {
+  if (internal::HasTrailingSlash(key)) {
     return true;
   }
   // Otherwise, if its content type starts with "application/x-directory",

Original file line number	Diff line number	Diff line change
`@@ -1221,7 +1221,7 @@ bool IsDirectory(std::string_view key, const S3Model::HeadObjectResult& result)`
`1221`	`1221`	`return false;`
`1222`	`1222`	`}`
`1223`	`1223`	`// Otherwise, if it has a trailing slash, it's a directory`
`1224`		`- if (key[key.size() - 1] == kSep) {`
	`1224`	`+ if (internal::HasTrailingSlash(key)) {`
`1225`	`1225`	`return true;`
`1226`	`1226`	`}`
`1227`	`1227`	`// Otherwise, if its content type starts with "application/x-directory",`