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...
Aside from this going against our code style guidelines where when calling a function that modifies a argument we want to have it syntactically obvious by putting an ampersand on it, this can be improves a bit.

DoJit is always using it's own code_buffer object, why not just remove this argument from this function call and just let it use the object directly?