Switch ImageUtils.cpp to use co_return (#12572)

rozele · web-flow · commit f358dc29db10 · 2024-01-09T14:07:17.000-08:00
* Switch ImageUtils.cpp to use co_return

In this async method, we should be using co_return. It appears MSVC is a bit more forgiving than clang on this front.

* Change files
diff --git a/change/react-native-windows-eef41231-d0eb-44a9-ac0d-d3905027d8bb.json b/change/react-native-windows-eef41231-d0eb-44a9-ac0d-d3905027d8bb.json
@@ -0,0 +1,7 @@
+{
+  "type": "prerelease",
+  "comment": "Switch ImageUtils.cpp to use co_return",
+  "packageName": "react-native-windows",
+  "email": "ericroz@meta.com",
+  "dependentChangeType": "patch"
+}
diff --git a/vnext/Microsoft.ReactNative/Utils/ImageUtils.cpp b/vnext/Microsoft.ReactNative/Utils/ImageUtils.cpp
@@ -60,7 +60,7 @@ winrt::IAsyncOperation<winrt::IRandomAccessStream> GetImageStreamAsync(ReactImag
       }
 
       winrt::Windows::Storage::Streams::IRandomAccessStreamWithContentType stream{openReadAsync.GetResults()};
-      return stream;
+      co_return stream;
     }
 
     auto httpMethod{

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ winrt::IAsyncOperation<winrt::IRandomAccessStream> GetImageStreamAsync(ReactImag`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`winrt::Windows::Storage::Streams::IRandomAccessStreamWithContentType stream{openReadAsync.GetResults()};`
`63`		`- return stream;`
	`63`	`+ co_return stream;`
`64`	`64`	`}`
`65`	`65`
`66`	`66`	`auto httpMethod{`