I believe the case of `err != nil` should be handled, particularly if the layout does not follow in lock step with the time format being written out. In that case old files will never be deleted?