Small refactorings in arrays.cpp and math.cpp

allot_array_1 and allot_array_2 are unused. And `factor_vm::unbox_array_size_slow`was written in a weird way which I think I've been able to simplify
Would it be better to add it to the end, so we don't force a VM recompile?  I'm not sure, but asking.