Hmm, it’s actually not easy at all to make this fail (I noticed while trying to write a test for this). The only thing where this double-call check is necessary is when `FileUtils.rm_f(filename)` fails, because in the next call, the function will exit before the `begin/rescue/end` block, because `!File.file?(filename)`.

So I think it’s OK to stick with the original implementation.