nitpick but it would be more obvious to just have 'DONE' instead of calling upper as well