Skip to content

Commit

Permalink
[10609] I consider it proven that all classes shall benefit from inte…
Browse files Browse the repository at this point in the history
…llect for weapon skill increase.

But better formula for chance would still welcome...
  • Loading branch information
Lynx3d committed Oct 14, 2010
1 parent 49a0d70 commit ae33712
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
5 changes: 1 addition & 4 deletions src/game/Player.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5587,10 +5587,7 @@ void Player::UpdateCombatSkills(Unit *pVictim, WeaponAttackType attType, bool de

float chance = float(3 * lvldif * skilldif) / plevel;
if(!defence)
{
if(getClass() == CLASS_WARRIOR || getClass() == CLASS_ROGUE)
chance *= 0.1f * GetStat(STAT_INTELLECT);
}
chance *= 0.1f * GetStat(STAT_INTELLECT);

chance = chance < 1.0f ? 1.0f : chance; //minimum chance to increase skill is 1%

Expand Down
2 changes: 1 addition & 1 deletion src/shared/revision_nr.h
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#ifndef __REVISION_NR_H__
#define __REVISION_NR_H__
#define REVISION_NR "10608"
#define REVISION_NR "10609"
#endif // __REVISION_NR_H__

5 comments on commit ae33712

@Thetruecrow
Copy link

Choose a reason for hiding this comment

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

Apparently, Intellect no longer serves this purpose.

@Lynx3d
Copy link
Contributor Author

@Lynx3d Lynx3d commented on ae33712 Oct 16, 2010

Choose a reason for hiding this comment

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

Says who?

@Revils
Copy link

@Revils Revils commented on ae33712 Oct 16, 2010

Choose a reason for hiding this comment

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

http://forums.worldofwarcraft.com/thread.html?topicId=15443424048&sid=1&pageNo=3
but I'm sure I read elsewhere that INT no longer serves for this.
Friend from offy told me that's common to use INT to level up weapon skills but he can't say if it's really proven or just a myth

@VladimirMangos
Copy link

Choose a reason for hiding this comment

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

As i understand this link related to 4.x? So not related to mangos master

@Revils
Copy link

@Revils Revils commented on ae33712 Oct 17, 2010

Choose a reason for hiding this comment

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

post n.40 is dated 03/09/2009 so it's for 3.x

Please sign in to comment.