Turn the square braces to round ones around that are around the regex. You are assigning a list ref to the first element of the list.

Also, I'm not too sure, but you may want to compile your regex outside of the subroutine. I may be wrong, but my instinks are telling me that qr!! will probably recompile the regex every time it is compiled.

Am I wrong?