IMO: 1) requiring a confusingly-named subroutine that doesn't take arguments, but rather must use global variables is awkward. 2) If you actually want to produce a list of filenames (which is a very common scenario), you need to do it yourself. 3) Changing the working directory by default is not very nice.