Log exception in GetEmulatorAvdNameAsync bare catch block

rmarinho · Copilot · rmarinho · commit 7801c18acef6 · 2026-03-05T12:51:56.000Z
Address review feedback: replace bare catch with catch(Exception ex)
and log via Trace.WriteLine for debuggability.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/src/Xamarin.Android.Tools.AndroidSdk/Runners/AdbRunner.cs b/src/Xamarin.Android.Tools.AndroidSdk/Runners/AdbRunner.cs
@@ -116,8 +116,8 @@ public async Task<IReadOnlyList<AdbDeviceInfo>> ListDevicesAsync (CancellationTo
 			}
 		} catch (OperationCanceledException) {
 			throw;
-		} catch {
-			// Silently ignore failures (emulator may not support this command)
+		} catch (Exception ex) {
+			Trace.WriteLine ($"GetEmulatorAvdNameAsync failed for '{serial}': {ex.Message}");
 		}
 
 		return null;

Original file line number	Diff line number	Diff line change
`@@ -116,8 +116,8 @@ public async Task<IReadOnlyList<AdbDeviceInfo>> ListDevicesAsync (CancellationTo`
`116`	`116`	`}`
`117`	`117`	`} catch (OperationCanceledException) {`
`118`	`118`	`throw;`
`119`		`- } catch {`
`120`		`- // Silently ignore failures (emulator may not support this command)`
	`119`	`+ } catch (Exception ex) {`
	`120`	`+ Trace.WriteLine ($"GetEmulatorAvdNameAsync failed for '{serial}': {ex.Message}");`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`return null;`