It's not. Please remove.  No need to waste more memory and have yet another variable, when the same check can be done on `sd->state.autoloottype` (it'll be zero when there are no items in the list, so it can be checked as a boolean just fine, as I did in the above code snippet at line 9360)