Skip to content

Commit

Permalink
[Hunter] Don't apply BM mastery to Guardian simulationcraft#4819
Browse files Browse the repository at this point in the history
Also removes Coordinated Assault from buffing Dire Consequences spawns
  • Loading branch information
Jayezi committed Jun 25, 2019
1 parent 587c02a commit de6fd03
Showing 1 changed file with 15 additions and 6 deletions.
21 changes: 15 additions & 6 deletions engine/class_modules/sc_hunter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -981,6 +981,16 @@ struct hunter_pet_t: public pet_t
main_hand_weapon.swing_time = 2.0_s;
}

double composite_player_multiplier( school_e school ) const override
{
double m = pet_t::composite_player_multiplier( school );

if ( o() -> mastery.master_of_beasts -> ok() )
m *= 1.0 + owner -> cache.mastery_value();

return m;
}

hunter_t* o() { return static_cast<hunter_t*>( owner ); }
const hunter_t* o() const { return static_cast<hunter_t*>( owner ); }
};
Expand Down Expand Up @@ -1142,6 +1152,7 @@ struct hunter_main_pet_base_t : public hunter_pet_t
double m = hunter_pet_t::composite_player_multiplier( school );

m *= 1.0 + buffs.bestial_wrath -> check_value();
m *= 1.0 + o() -> buffs.coordinated_assault -> check_value();

return m;
}
Expand Down Expand Up @@ -1442,7 +1453,10 @@ struct spitting_cobra_t: public hunter_pet_t
// for some reason it gets the player's multipliers
double composite_player_multiplier( school_e school ) const override
{
return owner -> composite_player_multiplier( school );
double m = owner -> composite_player_multiplier( school );
m *= 1.0 + owner -> cache.mastery_value();

return m;
}

void schedule_ready( timespan_t delta_time, bool waiting ) override
Expand Down Expand Up @@ -5624,17 +5638,12 @@ double hunter_t::composite_player_pet_damage_multiplier( const action_state_t* s
{
double m = player_t::composite_player_pet_damage_multiplier( s );

if ( mastery.master_of_beasts -> ok() )
m *= 1.0 + cache.mastery_value();

m *= 1.0 + specs.beast_mastery_hunter -> effectN( 3 ).percent();
m *= 1.0 + specs.survival_hunter -> effectN( 3 ).percent();
m *= 1.0 + specs.marksmanship_hunter -> effectN( 3 ).percent();

m *= 1.0 + talents.animal_companion -> effectN( 2 ).percent();

m *= 1.0 + buffs.coordinated_assault -> check_value();

return m;
}

Expand Down

0 comments on commit de6fd03

Please sign in to comment.