Flamethrower and messagesholelimit - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: Flamethrower and messagesholelimit (/showthread.php?tid=560955)

---

Flamethrower and messagesholelimit - Tamer - 31.01.2015

This is not a 0.3.7 issue. However, It is a major issue. If a player is using a flamethrower they can easily excess the messagesholelimit (which was 1000 for me, increasing the limit helps, but some spikes can cause it to exceed any higher value aswell) I highly recommend this issue to be checked and patched if possible at the next RC.

I did not test the persistence of this at 0.3.7

---

Re: Flamethrower and messagesholelimit - Slice - 31.01.2015

The issue is OnPlayerGiveDamage is being sent when you put someone on fire. If you put a few players on fire, you will flood the server with those updates.

I think fire should be excluded from OnPlayerGiveDamage (only happen in OnPlayerTakeDamage). It would resolve the issue with players breaching the message-hole limit.

---

Re: Flamethrower and messagesholelimit - Kar - 31.01.2015

I'm guessing we don't even need 'OnPlayerGiveDamage' anymore because of synced skin damage.

---

Re: Flamethrower and messagesholelimit - Tamer - 31.01.2015

Yes indeed, fire sources send way too many packets to the server. You can even do the demonstration of this by doing prints under that callback, the spam is outrageous.

---

AW: Re: Flamethrower and messagesholelimit - LadyNyuuu - 01.02.2015

We need that Callback for melee damage, because OnPlayerWeaponShot dont handle them.
OnPlayerGiveDamage and OnPlayerTakeDamage are important for clientside requests.

OnPlayerGiveDamage for killing someone with a baseballbat or fists while the other one is on desktop.
OnPlayerTakeDamage for request if someone got damage clientside like falldamage, hit by car or rotors.

---