Skip to content

Conversation

@Fairuz-Afdhal
Copy link

No description provided.

Micro-UZI weapon skill have a duplicate and sniper rifle is not added. This commit adds it.
@Fairuz-Afdhal
Copy link
Author

One of the problem with adding sniper rifle is that GetWeaponSkillID function must be modified like this. I can't think any other way to do it correctly.

stock GetWeaponSkillID(weaponid)
{
	if(weaponid == WEAPON_SNIPER) weaponid = 32;
	new index = weaponid - 22;
	if (!(0 <= index < sizeof(gWeaponSkills))) {
		return INVALID_WEAPON_SKILL_ID;
	}
	return gWeaponSkills[index];
}

@ziggi
Copy link
Member

ziggi commented Jul 22, 2021

I removed (1da4fa7) this some time ago because sniper skill doesn't works at all. Did you test it?

@Fairuz-Afdhal
Copy link
Author

Fairuz-Afdhal commented Jul 22, 2021

I removed (1da4fa7) this some time ago because sniper skill doesn't works at all. Did you test it?

Oh well okay, I never knew that lol! I'll close this right away.

EDIT: Just for curiosity, what do you mean that it doesn't work? do you mean it doesn't have any effect? Because IIRC it affects your accuracy for both Sniper Rifle and County Rifle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants