Source code clearly isn't canon - Ask any designer and they'll let you know such petty implementation details are beneath them. It's likely the coder just couldn't be bothered to do it properly, ie:
- Code:
static bool _god_hates_brand(const int brand)
Would contain the list of gods which actually hate various brands, while a second method:
- Code:
static bool _god_reminds_you_gently_that_this_brand_may_disrupt_your_mental_well_being_and_calm_serenity(const int brand)
would contain the line about Chei and chaos/speed brands. Then when coloring inventory (for example) you'd have to call both methods instead of just one, but that might allow you to do different coloring. For example, wands of hasting wouldn't have to be so violently red, they could be nice a pretty cyan or a tranquil green. Sure, it's messier for the coder to have to call both methods and not just one, so I understand why they'd take the shortcut. I just don't let it interfere with my enlightened understanding of time.
I may be taking this too far <.<