Clean up some warnings

This makes the build output length drops from 1500 lines to 300 in MSVC
Also, allocate the char with the right size from the start: `std::vector<u8> to_return(u16str.size());`. Then use indexing instead of `push_back` below.