Pop-up não obstrutiva

inseri no head:

<script type="text/javascript"><!--function janelaSecundaria (URL){window.open(URL,"popup","width=200,height=300,top=100,left=100,scrollbars=yes,resizable=yes")}function doPopups() {if (!document.getElementsByTagName) return false;var links = document.getElementsByTagName("a");for (var i=0; i < links.length; i++) { if (links[i].className.match("popup")) { links[i].onclick = janelaSecundaria(this.href); return false;}}}}window.onload = doPopups;//--></script>

e os links:

<a href="http://www.uol.com.br" class="popup">Link para um popup</a>

alguém pode me dizer o que está errado? não está funcionando..

o objetivo é ser um javascript não obstrutivo, por isso o endereço do link entra direto no <a>

valeu ;)