Use std::array in OpenXRUtils.cpp

AlexsanderDamaceno · cdumez · commit e5f71770a510 · 2025-03-23T00:03:10.000-07:00
https://bugs.webkit.org/show_bug.cgi?id=290199 Reviewed by Chris Dumez. Use std::array instead of hardcoded arrays * Source/WebCore/platform/xr/openxr/OpenXRUtils.h: (PlatformXR::resultToString): Canonical link: https://commits.webkit.org/292556@main
diff --git a/Source/WebCore/platform/xr/openxr/OpenXRUtils.h b/Source/WebCore/platform/xr/openxr/OpenXRUtils.h
@@ -50,10 +50,10 @@ T createStructure()
 
 inline String resultToString(XrResult value, XrInstance instance)
 {
-    char buffer[XR_MAX_RESULT_STRING_SIZE];
-    XrResult result = xrResultToString(instance, value, buffer);
+    std::array<char, XR_MAX_RESULT_STRING_SIZE> buffer;
+    XrResult result = xrResultToString(instance, value, buffer.data());
     if (result == XR_SUCCESS)
-        return String::fromLatin1(buffer);
+        return String::fromLatin1(buffer.data());
     return makeString("<unknown "_s, int(value), '>');
 }
 

Original file line number	Diff line number	Diff line change
`@@ -50,10 +50,10 @@ T createStructure()`
`50`	`50`
`51`	`51`	`inline String resultToString(XrResult value, XrInstance instance)`
`52`	`52`	`{`
`53`		`- char buffer[XR_MAX_RESULT_STRING_SIZE];`
`54`		`- XrResult result = xrResultToString(instance, value, buffer);`
	`53`	`+ std::array<char, XR_MAX_RESULT_STRING_SIZE> buffer;`
	`54`	`+ XrResult result = xrResultToString(instance, value, buffer.data());`
`55`	`55`	`if (result == XR_SUCCESS)`
`56`		`- return String::fromLatin1(buffer);`
	`56`	`+ return String::fromLatin1(buffer.data());`
`57`	`57`	`return makeString("<unknown "_s, int(value), '>');`
`58`	`58`	`}`
`59`	`59`