⚡ Optimize ToDictionary() in PointerInputDevice

[Dor-bl]

💡 What: Replaced the LINQ ToList().ForEach(...) iteration with a standard foreach loop inside PointerInputDevice.cs.

🎯 Why: The previous implementation called ToList() on a Dictionary, which creates an unnecessary List<KeyValuePair<string, object>> allocation. Using a standard foreach loop directly iterates the dictionary's enumerator without allocating extra memory, resulting in faster execution.

📊 Measured Improvement:
A focused benchmark mimicking the exact dictionary copy operation (with 100 extra attributes per iteration, tested over 100,000 iterations) yielded the following results locally:

• Baseline (LINQ ToList().ForEach): 788 ms
• Optimized (Standard foreach): 404 ms
• Improvement: ~48% reduction in execution time for this specific code path, alongside completely eliminating the intermediate list allocations.

---

PR created automatically by Jules for task 104955595590661477 started by @Dor-bl

[Copilot]

Pull request overview

This PR optimizes the payload merge logic in PointerInputDevice by replacing an allocation-heavy LINQ iteration with a direct dictionary enumeration, and adds a small standalone benchmark project to compare the two approaches.

Changes:

• Replaced ToList().ForEach(...) with a foreach loop when copying pointer extra attributes into the interaction payload.
• Added a benchmark/ console project to compare the LINQ vs foreach approaches.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
src/Appium.Net/Appium/Interactions/PointerInputDevice.cs	Removes intermediate list allocation by iterating the attributes dictionary directly.
benchmark/Program.cs	Adds an ad-hoc stopwatch benchmark comparing LINQ vs foreach.
benchmark/benchmark.csproj	Introduces a new benchmark console project and its target framework configuration.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.