Autopot delay

Quote:

Originally Posted by kevin_owner They are probably not using 1000ms between each potion but something higher like 1100 ms. You just have to find the sweet spot where you don't receive that error anymore. Another way of doing it would be to still use 1000 ms and check if you get the error message in a response. If so, send the potion casting packet again till it works.

Quote:

Originally Posted by lesderid Depending on your connection to the server, sending it, receiving the error and sending it again might take longer than simply finding the sweet spot (which is indeed probably around 1100 ms) and sending it once. Using potions as soon as possible can be quite crucial to keeping a character alive, so I would go for that solution.

bool busy = false;
byte SlotHP; // its value is set by UseHP

private void HandleHPUpdate(Packet packet)
{
	if (CurrentPercentHP <= ConfiguredPercentHP && !busy)
	{
		busy = true; // prevent concurrent operations
		UseHP(); // use potion now
	}
}

private void HandleItemUsed(Packet packet)
{
	if (response != 1) // failed to use item
	{
		// busy is never set back to false
		// because I have no idea which slot
		// it was trying to use
		return;
	}
	if (ItemSlotUsed == SlotHP && busy)
	{
		if (CurrentPercentHP <= ConfiguredPercentHP) // hp is still low
		{
			UseHP(1000); //use potion after 1000ms
		}
		else // hp is higher than configured, stop using potions
		{
			busy = false;
			SlotHP = 255;
		}
	}
}