Ecco una versione piu` corretta (forse e` anche senza errori)
codice:
function winopen1(ff){
  var nl = ff.newsletter; 
  if(!nl.value.match(/^\w+\@(\w+\.)+\w{2,4}$/) {
    alert("Inserisci un indirizzo e-mail valido!.");
    nl.focus();
    return false;
  }

  if (nl.value.length < 2) {
    alert("L'indirizzo di posta elettronica deve contenere almeno due caratteri!.")
    nl.focus();
    return false;
  }
  win1=window.open("", "click1", "history=no,resizable=no,status=no,scroll
bars=no,menubar=no,width=300,height=250");
}
Nota che la seconda parte e` inutile: dopo il filtro dato da quella RegExp, il campo non puo` essere di un carattere o vuoto.

A che ti serve aprire una finestra vuota? (mi riferisco all'ultima riga) Che ci metti dentro?

Con che istruzione chiami la funzione? Posta tutto il tag HTML con l'evento.