Windows: Old "with admin priviledges" version is broken

[dreua]

• On my win11 machine:
  • the default build with no admin privs does work as expected
  • the build with admin privs does not work. It only shows lost pings but with wireshark i could not see any icmp packets beeing sent. So that does not seem to work. I added some debug prints and sendto returns an expected value of 20. My windows debugging skills are at my limit there.

[...]

Even tho the version with admin privileges is broken i believe this PR is "completed" as on the current master this doesn't even compile. So i consider that feature "not made worse" :smile:

Originally posted by @mrbesen in https://github.com/cntools/cnping/issues/109#issuecomment-2102691518

I just wanted to have this documented, there is probably no real use in the "with admin prives" version anymore. Maybe we could consider removing the code and making the no admin version the only available.

[mrbesen]

Before we do remove that code, did someone other than me checked if it works? I have only tested it on one single machine, that might have some other (Group Policy) reasons for this to not work. Like i said, my windows debugging skills are limited.

But anyways: I am in favor of removing that code, as that is quite a lot of code, that is

• broken
• duplicating a feature (ping on windows)
• making the cnping src harder to read / understand
• adding a compile flag

[cnlohr]

Is there a recommended avenue forward, @dreua ?