Bukkit/Bukkit — comment on commit c80b2d2fd999edc682e855f9e110f23861e6c2e5
File:  line 0
By EdGruberman:

@Acrobot - Well actually I may be confusing something also. Or maybe it's @tahg that confused both of us? heh...

I just saw that @tahg does refer to the data field as a short. And I just looked closer and saw that ItemStack does not have a 3 parameter signature to pass in the data. It only has a 3 parameter signature to pass in damage. So what @tahg refers to as "data" here is actually "damage". As it stands this command will allow you to give potions out. LOL. However, it won't allow you to assign the data (MaterialData) as I believe @tahg intended?

But now that I look closer there is other funky things going on in ItemStack. Why does the data parameter in this signature: https://github.com/Bukkit/Bukkit/blob/c80b2d2fd999edc682e855f9e110f23861e6c2e5/src/main/java/org/bukkit/inventory/ItemStack.java#L46 override the damage/durability? That looks like a bug also. As I read it ItemStack is currently incapable of assigning data at instantiation due to this bug.

Am I totally not understanding something here myself?