well, since your using un-defined variables, the code would be invaild, but if you defined them elsewhere, then ok. but are the "?"'s used as part of your echo statement, or not? as that will affect it. also, you can actually make your entire statment in PHP, w/o having to type <?php ...?> for each part of it.
[code]<?php
<html>
<head>
<title>Example</title>
</head>
<body>
<p align="center">
echo 'Date: ".$todayis."/n';
echo 'Thank You : ".$visitor."( ".$visitormail." )/n';
echo 'Attention: ".$attn."/n';
echo "Message:";
$notesout = str_replace("\r", "<br/>", $notes);
echo $notesout;
echo $ip ;
</p>
?>