MSU's Individual Item Shop Script

[Saffith]

Link->Item[x] is already a bool, so there's no need to compare it against anything.

if(Link->Item[I_ITEM] && I_TYPE == 0)

Functionally, there's no difference between comparing a bool against true/false or 1/0; it's just more logical to compare two data of the same type.

[SUCCESSOR]

Link->Item[x] is already a bool, so there's no need to compare it against anything.

if(Link->Item[I_ITEM] && I_TYPE == 0)

Functionally, there's no difference between comparing a bool against true/false or 1/0; it's just more logical to compare two data of the same type.

Yes, this is what I was talking about. The redundancy only bothers me a little, but compairing a bool to 0 or 1 could be confusing to people and if you are sharing a script the less confusing and easier it is to read the better. I do prefer letting a bool stand on it's own. I mean, you did it with Link->Input but not Link->Item? Be consistant.