CG8516 / PvZA11y

An accessibility mod for Plants VS Zombies
MIT License
32 stars 11 forks source link

I have a merge request that fixes a bug #57

Open dellbeat opened 2 weeks ago

dellbeat commented 2 weeks ago

What's the bug?

If the Steam version of PVZ starts with popcapgame2 as the process name, the software will not execute as expected.

How to found the bug?

I have a computer running Windows 10. After updating it yesterday and opening PVZ, I tried to run this project but found that the process could not be recognized. Through program debugging and checking the information in Task Manager, I discovered that the launch process was popcapgame2, not popcapgame1.

Snipaste_2024-11-10_12-43-46

But when I thought it was an intermittent update of PVZ, I used another computer running Windows 11. In Steam online mode, I ran PVZ, and by checking in Task Manager, I found that the launch process was still popcapgame1.

Snipaste_2024-11-10_12-42-38

How to solve the bug?

Considering the existence of different process names, I added an array that includes the process names in the submission. I also handled the statements where appName needs to be checked for equality with constant values.

I will submit a merge request later.

dellbeat commented 2 weeks ago

Here are the pr