Looks like the character class [] vs grouping () has been answered well already .. That aside, I just wanted to recommend that you consider changing it even further to perhaps something like /\.(htm|asp)$/ to avoid presumably false hits on filenames like 'teaspoon.txt', 'grasp' and 'nightmare.txt' .. maybe use the /i modifier as well.