Loop not needed in Upload library

[eluser]Derek Jones[/eluser]
I'd have to go look at the source code to check, but I'm pretty sure str_replace() with an array does the same thing internally as that current code is doing in PHP, i.e. looping through an array and calling the actual replacement method over and over again. If memory serves, it even works on a copy of the array just like foreach() does. Not arguing against the code change, I agree it's simpler to just shove the array, but I don't think there's any performance or memory to be gained.