Remove a silly array wrapper class in factor of std::vector.

Could be refactored more heavily, but then again I'm scared of bounds checking vector implementations...
Because only part of the vector has actual code in it.  This could be changed in general to only `reserve`ing an appropriate amount of space and using `push_back` etc. to actually add ops, but I don't trust the vector to not free memory at inopportune times.