... since the [eE] means find zero or more e's or E's right?

No, it means one upper- or lower-case 'e'. Brackets mean a character class. You can make it accept zero with a question mark.

The simpler way to write that is with the /i modifier for a case-insensitive match:     return if ! /\.jpe?g$/i; I've made the 'e' optional in the match.

But that wouldn't find .jpeg

CountZero

"If you have four groups working on a compiler, you'll get a 4-pass compiler." - Conway's Law

Golly, you're quick! It didn't take me a minute from posting to fix that ;-)

After Compline,
Zaxo

return if ! ( /\.[jJ][pP][eE]?[gG]$/ );
[download]

The brackets mean "one of these characters" while the question mark means "the preceeding element is optional" or "0 or 1 of the preceeding element", depending on how you want to look at it.

Of course, I'd switch out the "if !" for "unless"... but there be another religious war ;-)

(PS - unless performance matters, just use the i modifier - /\.jpe?g/i is just about as fast as the above.)

since the [eE] means find zero or more e's or E's right?

Actually not quite. [Ee] means find exactly one e or E. [Ee]* means find zero or more, whereas [Ee]? means find either zero or one.

By the way, another trick you may find useful is case-insensitive matching. So actually you can just say:

return if ! (/\.jpe?g$/i );
[download]

Take a look at perlretut and Common Regex Gotchas.