I meant exactly the opposite. We should keep the old one because I do not see the reason for this "do all" new function. Clean-up has to be left to logrotate.

Would you explain why we do need this new function in lieu of the previous one? Otherwise I would stick with the previous one and take the chance to remove the ASSERT, too