Two bugs with Glorfindel my brother and me noticed:
1.) Glorfindel's Blade of Purity supposedly makes him "take only half the damage" from the abilities of enemy heroes.
In truth, it simply makes him invulnerable to magic damage.
(The reason for this is that he has "Invulnerability 50% MAGIC" in the ModifierList
GlorfindelBladeOfPurityM od. The number behind invulnerability does not matter, it is always full immunity.)
I think the ability should be reworded from "abilities of heroes" to "Magic damage" and the invulnerability be changed to ARMOR 50% (or kept the way it is and also incorporated into the description).
2.) The ability "Brilliant Illumination" is a bit of a mess, including a slightly misleading description:
"Glorfindel is the only Elf who has seen the Halls of Mandos and returned to Middle-earth, bringing the blessing of the Valar with him. Every 15 seconds he will be completely protected from an enemy attack or enemy magic."
In reality, it only triggers when an enemy hero is in his close vicinity. Also, it gives him invulnerability to magic for 2.5 seconds, so it would indeed defend him from all magic attacks, but simultaneously gives him 200% armour which of course is no "complete protection". Therefore, nonmagical attacks against him will not be averted completely.
This however probably doesn't matter, as the Modifier "GlorfindelSelfBlock" also includes "Modifier = ARMOR 0% MAGIC" which by itself does nothing; but I'm pretty sure it overrites the earlier ARMOR Modifier.
My suggestion to fix this would be removing it completely :P
(As a sidenote, Level 7 seems a bit late for this ability, especially since it's so situational - the armour buff is useless if it triggers while Aragorn approaches and turns on Blademaster but then wears off before Aragorn lays into Glorfindel a few times. Not to mention BladeOfPurity already does half of this ability's job.)
-OTWK