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
I think this needs to be a ``data_root``, since the next call is ``allot_array`` which might cause GC?