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

Wind Deflection - Only track remote bullets if they are tracers #5839

Merged
merged 1 commit into from
Dec 2, 2017

Conversation

ulteq
Copy link
Contributor

@ulteq ulteq commented Dec 2, 2017

  • Do not run the PFH for remote projectiles unless they are tracers

* Do not run the PFH for remote projectiles unless they are tracers
@ulteq ulteq added the kind/optimization Release Notes: **IMPROVED:** label Dec 2, 2017
@ulteq ulteq added this to the 3.12.0 milestone Dec 2, 2017
@ulteq ulteq merged commit b7be7d1 into master Dec 2, 2017
@ulteq ulteq deleted the winddeflection-optimization branch December 2, 2017 13:18

if (!((_projectile isKindOf "BulletBase") || {_projectile isKindOf "GrenadeBase"})) exitWith {false};
if (_unit distance ACE_player > GVAR(simulationRadius)) exitWith {false};

private _abort = false;
if (!local _unit && {_projectile isKindOf "BulletBase"}) then {
private _ammoCount = (_unit ammo _muzzle) + 1;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just adding a note, don't think it will be a problem
This function is called for vehicles as well and ammo doesn't seem to work for non-primary turret.
e.g. Right door gunner of ghost hawk

(vehicle player) ammo "LMG_Minigun_Transport2" is always 0

BaerMitUmlaut pushed a commit that referenced this pull request Aug 5, 2019
* Do not run the PFH for remote projectiles unless they are tracers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/optimization Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants