Tree drops likely not consistent with vanilla behavior

[garrettjoecox]

I came across this while messing with tree bonking behavior in the holiday build, in #3171 an additional branch was added, leaving a hanging additional else condition, I'm honestly surprised it compiles and not really sure how it behaves in practice, but we should fix this, and maybe even VB-ify the behavior.

soh/src/overlays/actors/ovl_En_Wood02/z_en_wood02.c

if ((this->unk_14C >= 0) && (this->unk_14C < 0x64) && (CVarGetInteger("gTreeStickDrops", 0)) && !(INV_CONTENT(ITEM_STICK) == ITEM_NONE)) {
    ...
} else {
    ...
} else {
    ...
}

[serprex]

It compiles because the code is poorly indented, within the else there's an if that corresponds to last else