Как уже было сказано, вам нужно удалить разрыв;
Однако вам также нужно беспокоиться о области действия переменной i.
Проблема заключается в том, что меня ставят до 6, а затем, когда вещи впоследствии называют индексными ссылками [6], которых не существует, искусственно решая эту проблему для первой ссылки.
Следующее исправит вашу непосредственную проблему:
links[i].onclick =
function() { var idx=i; return function(){ popup(links[idx]);} }();
, сохраняя при этом весь исходный код.