As specified in the Docs this hook has "No return behaviour", and has a return type "void", but in the Hammer class is obviously specified, that DoAttackShared checks for non-null return type before applying the attack. Fix the docs mb?

        public override void DoAttackShared(HitInfo info)
	{
		BasePlayer ownerPlayer = GetOwnerPlayer();
		BaseCombatEntity baseCombatEntity = info.HitEntity as BaseCombatEntity;
		if (baseCombatEntity != null)
		{
			if (Interface.CallHook("OnHammerHit", ownerPlayer, info) != null)
			{
				return;
			}
			if (ownerPlayer != null && base.isServer)
			{
				using (TimeWarning.New("DoRepair", 50L))
				{
					baseCombatEntity.DoRepair(ownerPlayer);
				}
			}
		}
		if (base.isServer)
		{
			Effect.server.ImpactEffect(info);
		}
		else
		{
			Effect.client.ImpactEffect(info);
		}
	}

There are a lot of that kind of functions, thats why i prefer check it before use

Not aware of any others they are incorrect, perhaps you can enlighten us? If we aren’t aware of something, we can’t really fix it.

In uncorrect i mean that they are object-based, it really will be a problem to grab them all, but i will try