Pues lo de la codificación yo tampoco lo tengo muy claro (es uno de esos temas que siempre tengo pendiente de mirar

), pero si el validador no dice nada al respecto, supongo que será correcto

siempre que todos los agentes de usuario que visualicen la página, sean compatibles.
En cuanto al doctype, ya se ha hablado mucho de ello. Bien es cierto que hay que tender, con el paso del tiempo, a usar la última versión, y de la forma más estricta (para tender a que el documento sea auténtico XML), pero no es una obligación ni se es "más mejor" :P Simplemente hay que usar el que más convenga (que para eso existen

): Por ejemplo, XHTML 1.0 strict no permite el atributo "target" en los enlaces, pero el transitional sí. Eso no quiere decir que uno sea mejor o peor que el otro, depende de tus necesidades actuales.
Bueno, y ya por último, lo del
... No me había fijado que era para separar los enlaces. Supongo que en ese caso se puede dar por bueno, ya que aunque visualmente se pueden emplear márgenes, eso no se toma como caracteres imprimibles, aunque otra solución es emplear listas sin numerar, formateadas correctamente con CSS, ya que es la forma más común de agrupar enlaces... bueno, eso ya va a gusto del consumidor
Un saludo!