But it won't work, because if `sd->state.autoloottype == ITEM_MAX` is true, then also `sd->state.autoloottype&(1<<type)` is true, so it was already caught on line 5875 and returned on line 5877.

It's not really necessary to tell the user that all the types are autolooted in any case, since you're already covering it with `You're already autolooting this item type.` anyways, this seems a bit too redundant to me (and a waste of a msg_txt string)