Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default ping value #98

Open
ziggi opened this issue Jul 18, 2017 · 6 comments
Open

Default ping value #98

ziggi opened this issue Jul 18, 2017 · 6 comments

Comments

@ziggi
Copy link
Owner

ziggi commented Jul 18, 2017

Default ping value is -1, this should be 0 if it's possible.

@rt-2
Copy link

rt-2 commented Jul 18, 2017

Why?
NPCs don't have ping right? Why assign them a possible value?

@WoutProvost
Copy link
Contributor

When I use /rcon players, I get this:
Ping and IP
1 to 5 are standard NPCs, 6 is me, and 46 to 49 are FCNPCs. Notice the that the IP is different as well, not that it really bothers anyone I think though.

@rt-2
Copy link

rt-2 commented Jul 18, 2017

I still don't see the problem....
I hope I'm not getting on your nerves

@WoutProvost
Copy link
Contributor

Me neither, however I can think of something like this:

new ping = GetPlayerPing(playerid);
if(ping < 0 || ping == 65535) {
	//Ping is invalid
} else {
	//Ping is valid
}

@NegativeIQ
Copy link

0ms is possible and actually highly likely in some configurations and high end machines.
-1 is impossible ping so i guess it should stay -1.

But in any case its easy to hook and return 0 in pawn instead of -1 ....

ziggi added a commit that referenced this issue Nov 4, 2017
@ziggi
Copy link
Owner Author

ziggi commented Nov 4, 2017

Done.

@ziggi ziggi closed this as completed Nov 4, 2017
ziggi added a commit that referenced this issue Jan 7, 2018
ziggi added a commit that referenced this issue Jan 7, 2018
@ziggi ziggi reopened this Jan 7, 2018
@ziggi ziggi added the 2.0 label Jan 7, 2018
@ziggi ziggi added this to the FCNPC 2.0 milestone Jul 10, 2018
@ziggi ziggi mentioned this issue Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants